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

Revision history for this message
Neil J. Patel (njpatel) wrote :

Looks good. Just one question:

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?

review: Needs Information

« Back to merge proposal