Code review comment for lp:~attente/unity-gtk-module/1243413

Revision history for this message
William Hua (attente) wrote :

Thanks Charles!

is_blacklisted is only called once when the module is first loaded. I'm thinking it's probably better to unref the GSettings object as soon as it's done with it rather than keeping it around for the life of the process. If you still disagree though, I'll fix it.

But agreed on your other points, hopefully those are fixed now.

« Back to merge proposal