Review: Approve
180     +       if (seendb != NULL) {
181     +               return;
182     +       }

I kind of wouldn't mind seeing that change to a g_return_if_fail(), because it 
seems like if you are hitting this multiple times there's probably something 
screwy going on right?

Approve.
-- 
https://code.edge.launchpad.net/~ted/indicator-messages/seen-applications-list/+merge/22100
Your team ayatana-commits is subscribed to branch lp:indicator-messages.

_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp

Reply via email to