Code review comment for lp:~ted/indicator-messages/menu-update

Revision history for this message
Ted Gould (ted) wrote :

On Thu, 2009-08-27 at 14:20 +0000, Neil J. Patel wrote:
> In new_launcher_item; I know it's a static function, so was
> just wondering if args were checked before calling it
> elsewhere in the code, or should the function also check
> it's args just in case?

Remember that comment on the dbusmenu review about application
developers stealing code and not thinking about it... I don't know any
of those ;)

Fixed (r141)

« Back to merge proposal