Code review comment for lp:~ted/libdbusmenu/now-with-parsing

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

 review needsfixing

Just two minor things; dbusmenu_gtk_parse_menu_structure() needs
documentation. In order to help would/be consumers with debugging you
should also add a g_return_val_if_fail(GTK_IS_MENU_ITEM(widget) ||
GTK_IS_MENU_SHELL (widget),NULL).

review: Needs Fixing

« Back to merge proposal