diff options
author | Ken VanDine <ken.vandine@canonical.com> | 2010-07-08 17:18:49 -0400 |
---|---|---|
committer | Ken VanDine <ken.vandine@canonical.com> | 2010-07-08 17:18:49 -0400 |
commit | 271f62baede153472711c3dcf2388df3acffa006 (patch) | |
tree | 0432b0802a7b35beaac76d51d797219a4b586d80 /libdbusmenu-gtk | |
parent | 1231c20212abb9c09b9eaec0c8619d126e3e2779 (diff) | |
parent | 78b810a93fbccde56192efa7cc6b4a520f03e35a (diff) | |
download | libdbusmenu-271f62baede153472711c3dcf2388df3acffa006.tar.gz libdbusmenu-271f62baede153472711c3dcf2388df3acffa006.tar.bz2 libdbusmenu-271f62baede153472711c3dcf2388df3acffa006.zip |
releasing version 0.3.5-0ubuntu1
Diffstat (limited to 'libdbusmenu-gtk')
-rw-r--r-- | libdbusmenu-gtk/client.c | 2 | ||||
-rw-r--r-- | libdbusmenu-gtk/menuitem.c | 7 |
2 files changed, 6 insertions, 3 deletions
diff --git a/libdbusmenu-gtk/client.c b/libdbusmenu-gtk/client.c index a46aef3..b406697 100644 --- a/libdbusmenu-gtk/client.c +++ b/libdbusmenu-gtk/client.c @@ -148,7 +148,9 @@ do_swap_agroup (DbusmenuMenuitem * mi, gpointer userdata) { return FALSE; } + #ifdef MASSIVEDEBUGGING g_debug("Setting shortcut on '%s': %d %X", dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_LABEL), key, modifiers); + #endif GtkMenuItem * gmi = dbusmenu_gtkclient_menuitem_get(data->client, mi); if (gmi == NULL) { diff --git a/libdbusmenu-gtk/menuitem.c b/libdbusmenu-gtk/menuitem.c index 9924546..5846aa7 100644 --- a/libdbusmenu-gtk/menuitem.c +++ b/libdbusmenu-gtk/menuitem.c @@ -264,9 +264,10 @@ dbusmenu_menuitem_property_set_shortcut_menuitem (DbusmenuMenuitem * menuitem, c GtkAccelGroup * group = gtk_accel_group_from_accel_closure(closure); - /* Seriously, if this returns NULL something is seriously - wrong in GTK. */ - g_return_val_if_fail(group != NULL, FALSE); + /* Apparently this is more common than I thought. */ + if (group == NULL) { + return FALSE; + } GtkAccelKey * key = gtk_accel_group_find(group, find_closure, closure); /* Again, not much we can do except complain loudly. */ |