From ebace67d14edcd8e0e4d3d1ebe5e54e2b59270b3 Mon Sep 17 00:00:00 2001 From: Nick Dedekind Date: Tue, 27 Aug 2013 17:24:38 +0100 Subject: Review comments --- libqmenumodel/src/gtk/gtkactionobserveritem.c | 170 ------------------------ libqmenumodel/src/gtk/gtkactionobserveritem.h | 67 ---------- libqmenumodel/src/gtk/gtksimpleactionobserver.c | 160 ++++++++++++++++++++++ libqmenumodel/src/gtk/gtksimpleactionobserver.h | 67 ++++++++++ 4 files changed, 227 insertions(+), 237 deletions(-) delete mode 100644 libqmenumodel/src/gtk/gtkactionobserveritem.c delete mode 100644 libqmenumodel/src/gtk/gtkactionobserveritem.h create mode 100644 libqmenumodel/src/gtk/gtksimpleactionobserver.c create mode 100644 libqmenumodel/src/gtk/gtksimpleactionobserver.h (limited to 'libqmenumodel/src/gtk') diff --git a/libqmenumodel/src/gtk/gtkactionobserveritem.c b/libqmenumodel/src/gtk/gtkactionobserveritem.c deleted file mode 100644 index db4c544..0000000 --- a/libqmenumodel/src/gtk/gtkactionobserveritem.c +++ /dev/null @@ -1,170 +0,0 @@ -/* - * Copyright © 2013 Canonical Limited - * - * This library is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License as - * published by the Free Software Foundation; either version 2 of the - * licence or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library. If not, see . - * - * Authors: Nick Dedekind observable) - g_object_unref (self->observable); - - if (self->action_name) - g_free(self->action_name); - - G_OBJECT_CLASS (gtk_action_observer_item_parent_class)->finalize (object); -} - -static void -gtk_action_observer_item_init (GtkActionObserverItem * self) -{ -} - -static void -gtk_action_observer_item_class_init (GtkActionObserverItemClass *class) -{ - class->finalize = gtk_action_observer_item_finalize; -} - -static void -gtk_action_observer_item_action_added (GtkActionObserver *observer, - GtkActionObservable *observable, - const gchar *action_name, - const GVariantType *parameter_type, - gboolean enabled, - GVariant *state) -{ - g_return_if_fail (GTK_IS_ACTION_OBSERVER_ITEM (observer)); - - GtkActionObserverItem* self; - self = GTK_ACTION_OBSERVER_ITEM (observer); - self->action_added(self, action_name, enabled, state); -} - -static void -gtk_action_observer_item_action_enabled_changed (GtkActionObserver *observer, - GtkActionObservable *observable, - const gchar *action_name, - gboolean enabled) -{ - g_return_if_fail (GTK_IS_ACTION_OBSERVER_ITEM (observer)); - - GtkActionObserverItem* self; - self = GTK_ACTION_OBSERVER_ITEM (observer); - self->action_enabled_changed(self, action_name, enabled); -} - -static void -gtk_action_observer_item_action_state_changed (GtkActionObserver *observer, - GtkActionObservable *observable, - const gchar *action_name, - GVariant *state) -{ - g_return_if_fail (GTK_IS_ACTION_OBSERVER_ITEM (observer)); - - GtkActionObserverItem* self; - self = GTK_ACTION_OBSERVER_ITEM (observer); - self->action_state_changed(self, action_name, state); -} - -static void -gtk_action_observer_item_action_removed (GtkActionObserver *observer, - GtkActionObservable *observable, - const gchar *action_name) -{ - g_return_if_fail (GTK_IS_ACTION_OBSERVER_ITEM (observer)); - - GtkActionObserverItem* self; - self = GTK_ACTION_OBSERVER_ITEM (observer); - self->action_removed(self, action_name); -} - -static void -gtk_action_observer_item_init_observer_iface (GtkActionObserverInterface *iface) -{ - iface->action_added = gtk_action_observer_item_action_added; - iface->action_enabled_changed = gtk_action_observer_item_action_enabled_changed; - iface->action_state_changed = gtk_action_observer_item_action_state_changed; - iface->action_removed = gtk_action_observer_item_action_removed; -} - -GtkActionObserverItem* -gtk_action_observer_item_new (GtkActionObservable *observable, - GtkActionAddedFunc action_added, - GtkActionEnabledChangedFunc action_enabled_changed, - GtkActionStateChangedFunc action_state_changed, - GtkActionRemovedFunc action_removed) -{ - GtkActionObserverItem* self; - self = g_object_new (GTK_TYPE_ACTION_OBSERVER_ITEM, NULL); - self->observable = g_object_ref (observable); - self->action_name = NULL; - - self->action_added = action_added; - self->action_enabled_changed = action_enabled_changed; - self->action_state_changed = action_state_changed; - self->action_removed = action_removed; - - return self; -} - -void -gtk_action_observer_item_register_action (GtkActionObserverItem *self, - const gchar* action_name) -{ - gtk_action_observer_item_unregister_action(self); - - if (action_name && g_strcmp0(action_name, "") != 0) { - self->action_name = g_strdup (action_name); - - if (g_strcmp0(self->action_name, "") != 0) { - gtk_action_observable_register_observer (self->observable, self->action_name, GTK_ACTION_OBSERVER (self)); - } - } -} - -void -gtk_action_observer_item_unregister_action (GtkActionObserverItem *self) -{ - if (self->action_name) { - gtk_action_observable_unregister_observer(self->observable, self->action_name, GTK_ACTION_OBSERVER (self)); - g_free(self->action_name); - self->action_name = NULL; - } -} diff --git a/libqmenumodel/src/gtk/gtkactionobserveritem.h b/libqmenumodel/src/gtk/gtkactionobserveritem.h deleted file mode 100644 index 315cadc..0000000 --- a/libqmenumodel/src/gtk/gtkactionobserveritem.h +++ /dev/null @@ -1,67 +0,0 @@ -/* - * Copyright © 2013 Canonical Limited - * - * This library is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License as - * published by the Free Software Foundation; either version 2 of the - * licence or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library. If not, see . - * - * Authors: Nick Dedekind . + * + * Authors: Nick Dedekind observable); + + g_free(self->action_name); + + G_OBJECT_CLASS (gtk_simple_action_observer_parent_class)->finalize (object); +} + +static void +gtk_simple_action_observer_init (GtkSimpleActionObserver * self) +{ +} + +static void +gtk_simple_action_observer_class_init (GtkSimpleActionObserverClass *class) +{ + class->finalize = gtk_simple_action_observer_finalize; +} + +static void +gtk_simple_action_observer_action_added (GtkActionObserver *observer, + GtkActionObservable *observable, + const gchar *action_name, + const GVariantType *parameter_type, + gboolean enabled, + GVariant *state) +{ + GtkSimpleActionObserver* self; + self = GTK_SIMPLE_ACTION_OBSERVER (observer); + self->action_added(self, action_name, enabled, state); +} + +static void +gtk_simple_action_observer_action_enabled_changed (GtkActionObserver *observer, + GtkActionObservable *observable, + const gchar *action_name, + gboolean enabled) +{ + GtkSimpleActionObserver* self; + self = GTK_SIMPLE_ACTION_OBSERVER (observer); + self->action_enabled_changed(self, action_name, enabled); +} + +static void +gtk_simple_action_observer_action_state_changed (GtkActionObserver *observer, + GtkActionObservable *observable, + const gchar *action_name, + GVariant *state) +{ + GtkSimpleActionObserver* self; + self = GTK_SIMPLE_ACTION_OBSERVER (observer); + self->action_state_changed(self, action_name, state); +} + +static void +gtk_simple_action_observer_action_removed (GtkActionObserver *observer, + GtkActionObservable *observable, + const gchar *action_name) +{ + GtkSimpleActionObserver* self; + self = GTK_SIMPLE_ACTION_OBSERVER (observer); + self->action_removed(self, action_name); +} + +static void +gtk_simple_action_observer_init_observer_iface (GtkActionObserverInterface *iface) +{ + iface->action_added = gtk_simple_action_observer_action_added; + iface->action_enabled_changed = gtk_simple_action_observer_action_enabled_changed; + iface->action_state_changed = gtk_simple_action_observer_action_state_changed; + iface->action_removed = gtk_simple_action_observer_action_removed; +} + +GtkSimpleActionObserver* +gtk_simple_action_observer_new (GtkActionObservable *observable, + GtkActionAddedFunc action_added, + GtkActionEnabledChangedFunc action_enabled_changed, + GtkActionStateChangedFunc action_state_changed, + GtkActionRemovedFunc action_removed) +{ + GtkSimpleActionObserver* self; + self = g_object_new (GTK_TYPE_SIMPLE_ACTION_OBSERVER, NULL); + self->observable = g_object_ref (observable); + self->action_name = NULL; + + self->action_added = action_added; + self->action_enabled_changed = action_enabled_changed; + self->action_state_changed = action_state_changed; + self->action_removed = action_removed; + + return self; +} + +void +gtk_simple_action_observer_register_action (GtkSimpleActionObserver *self, + const gchar *action_name) +{ + gtk_simple_action_observer_unregister_action(self); + + if (action_name && g_strcmp0(action_name, "") != 0) { + self->action_name = g_strdup (action_name); + + if (g_strcmp0(self->action_name, "") != 0) { + gtk_action_observable_register_observer (self->observable, self->action_name, GTK_ACTION_OBSERVER (self)); + } + } +} + +void +gtk_simple_action_observer_unregister_action (GtkSimpleActionObserver *self) +{ + if (self->action_name) { + gtk_action_observable_unregister_observer(self->observable, self->action_name, GTK_ACTION_OBSERVER (self)); + g_free(self->action_name); + self->action_name = NULL; + } +} diff --git a/libqmenumodel/src/gtk/gtksimpleactionobserver.h b/libqmenumodel/src/gtk/gtksimpleactionobserver.h new file mode 100644 index 0000000..4276786 --- /dev/null +++ b/libqmenumodel/src/gtk/gtksimpleactionobserver.h @@ -0,0 +1,67 @@ +/* + * Copyright © 2013 Canonical Limited + * + * This library is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation; either version 2 of the + * licence or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library. If not, see . + * + * Authors: Nick Dedekind