← Back to team overview

ayatana-commits team mailing list archive

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