Merge lp:~ted/libdbusmenu/lp940651 into lp:libdbusmenu/0.6
Proposed by
Ted Gould
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 378 | ||||
Proposed branch: | lp:~ted/libdbusmenu/lp940651 | ||||
Merge into: | lp:libdbusmenu/0.6 | ||||
Diff against target: |
117 lines (+49/-47) 1 file modified
libdbusmenu-glib/client.c (+49/-47) |
||||
To merge this branch: | bzr merge lp:~ted/libdbusmenu/lp940651 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Kerr (community) | Approve | ||
DBus Menu Team | Pending | ||
Review via email: mp+95413@code.launchpad.net |
Description of the change
Fix some goto cleanups so things are more explicit
To post a comment you must log in.
Approved because the code looks fine, but I have a two minor suggestions for cleanup:
1. g_clear_ error(& localerror) instead of "if (localerror != NULL) { g_error_ free(localerror ); }
2. The "callback all the folks we can find" section can be simpler:
properties_ listener_ t * listener = find_listener( listeners, 0, id);
g_warning( "Unable to find listener for ID %d", id);
g_warning( "Odd, we've already replied to the listener on ID %d", id); get_child_ value(child, 1);
listener- >callback( properties, NULL, listener- >user_data) ;
listener- >replied = TRUE;
g_variant_ unref(propertie s);
if (listener == NULL) {
} else if (listener->replied) {
} else {
GVariant * properties = g_variant_
}
g_variant_ unref(child) ;