ayatana-commits team mailing list archive
-
ayatana-commits team
-
Mailing list archive
-
Message #01294
Re: [Merge] lp:~ted/indicator-messages/seen-applications-list into lp:indicator-messages
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.
References