From 5ac61cc67e3874547b83f5c05ae2c1b76a6de3bb Mon Sep 17 00:00:00 2001 From: Matthew Barnes Date: Wed, 27 Mar 2013 20:33:42 -0400 Subject: ESelectNamesRenderer cleanups. --- calendar/gui/e-select-names-renderer.c | 253 ++++++++++++++++++++------------- calendar/gui/e-select-names-renderer.h | 15 +- 2 files changed, 170 insertions(+), 98 deletions(-) (limited to 'calendar') diff --git a/calendar/gui/e-select-names-renderer.c b/calendar/gui/e-select-names-renderer.c index 02a21ac2e2..8b8f20abad 100644 --- a/calendar/gui/e-select-names-renderer.c +++ b/calendar/gui/e-select-names-renderer.c @@ -61,22 +61,26 @@ G_DEFINE_TYPE ( static void e_select_names_renderer_editing_done (GtkCellEditable *editable, - ESelectNamesRenderer *cell) + ESelectNamesRenderer *renderer) { GList *addresses = NULL, *names = NULL, *a, *n; gboolean editing_canceled; /* We don't need to listen for the focus out event any more */ - g_signal_handlers_disconnect_matched (editable, G_SIGNAL_MATCH_DATA, 0, 0, NULL, NULL, cell); + g_signal_handlers_disconnect_matched ( + editable, G_SIGNAL_MATCH_DATA, 0, 0, NULL, NULL, renderer); g_object_get (editable, "editing-canceled", &editing_canceled, NULL); if (editing_canceled) { - gtk_cell_renderer_stop_editing (GTK_CELL_RENDERER (cell), TRUE); + gtk_cell_renderer_stop_editing ( + GTK_CELL_RENDERER (renderer), TRUE); goto cleanup; } - addresses = e_select_names_editable_get_emails (E_SELECT_NAMES_EDITABLE (editable)); - names = e_select_names_editable_get_names (E_SELECT_NAMES_EDITABLE (editable)); + addresses = e_select_names_editable_get_emails ( + E_SELECT_NAMES_EDITABLE (editable)); + names = e_select_names_editable_get_names ( + E_SELECT_NAMES_EDITABLE (editable)); /* remove empty addresses */ for (a = addresses, n = names; a && n;) { @@ -95,27 +99,105 @@ e_select_names_renderer_editing_done (GtkCellEditable *editable, } } - g_signal_emit (cell, signals[CELL_EDITED], 0, cell->priv->path, addresses, names); + g_signal_emit ( + renderer, signals[CELL_EDITED], 0, + renderer->priv->path, addresses, names); - g_list_foreach (addresses, (GFunc) g_free, NULL); - g_list_foreach (names, (GFunc) g_free, NULL); - g_list_free (addresses); - g_list_free (names); + g_list_free_full (addresses, (GDestroyNotify) g_free); + g_list_free_full (names, (GDestroyNotify) g_free); - cleanup: - g_free (cell->priv->path); - cell->priv->path = NULL; - cell->priv->editable = NULL; +cleanup: + g_free (renderer->priv->path); + renderer->priv->path = NULL; + renderer->priv->editable = NULL; +} + +static void +select_names_renderer_set_property (GObject *object, + guint property_id, + const GValue *value, + GParamSpec *pspec) +{ + switch (property_id) { + case PROP_NAME: + e_select_names_renderer_set_name ( + E_SELECT_NAMES_RENDERER (object), + g_value_get_string (value)); + return; + + case PROP_EMAIL: + e_select_names_renderer_set_email ( + E_SELECT_NAMES_RENDERER (object), + g_value_get_string (value)); + return; + } + + G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec); +} + +static void +select_names_renderer_get_property (GObject *object, + guint property_id, + GValue *value, + GParamSpec *pspec) +{ + switch (property_id) { + case PROP_NAME: + g_value_set_string ( + value, + e_select_names_renderer_get_name ( + E_SELECT_NAMES_RENDERER (object))); + return; + + case PROP_EMAIL: + g_value_set_string ( + value, + e_select_names_renderer_get_email ( + E_SELECT_NAMES_RENDERER (object))); + return; + } + + G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec); +} + +static void +select_names_renderer_dispose (GObject *object) +{ + ESelectNamesRendererPrivate *priv; + + priv = E_SELECT_NAMES_RENDERER_GET_PRIVATE (object); + + g_clear_object (&priv->editable); + + /* Chain up to parent's dispose() method. */ + G_OBJECT_CLASS (e_select_names_renderer_parent_class)-> + dispose (object); +} + +static void +select_names_renderer_finalize (GObject *object) +{ + ESelectNamesRendererPrivate *priv; + + priv = E_SELECT_NAMES_RENDERER_GET_PRIVATE (object); + + g_free (priv->path); + g_free (priv->name); + g_free (priv->email); + + /* Chain up to parent's finalize() method. */ + G_OBJECT_CLASS (e_select_names_renderer_parent_class)-> + finalize (object); } static GtkCellEditable * -e_select_names_renderer_start_editing (GtkCellRenderer *cell, - GdkEvent *event, - GtkWidget *widget, - const gchar *path, - const GdkRectangle *bg_area, - const GdkRectangle *cell_area, - GtkCellRendererState flags) +select_names_renderer_start_editing (GtkCellRenderer *cell, + GdkEvent *event, + GtkWidget *widget, + const gchar *path, + const GdkRectangle *bg_area, + const GdkRectangle *cell_area, + GtkCellRendererState flags) { ESelectNamesRenderer *sn_cell = E_SELECT_NAMES_RENDERER (cell); GtkCellRendererText *text_cell = GTK_CELL_RENDERER_TEXT (cell); @@ -134,7 +216,7 @@ e_select_names_renderer_start_editing (GtkCellRenderer *cell, editable = e_select_names_editable_new (); gtk_entry_set_has_frame (GTK_ENTRY (editable), FALSE); gtk_entry_set_alignment (GTK_ENTRY (editable), xalign); - if (sn_cell->priv->email && *sn_cell->priv->email) + if (sn_cell->priv->email != NULL && *sn_cell->priv->email != '\0') e_select_names_editable_set_address ( E_SELECT_NAMES_EDITABLE (editable), sn_cell->priv->name, @@ -145,91 +227,28 @@ e_select_names_renderer_start_editing (GtkCellRenderer *cell, editable, "editing_done", G_CALLBACK (e_select_names_renderer_editing_done), sn_cell); - /* Removed focus-out-event. focus out event already listen by base class. - * We don't need to listen for the focus out event any more */ - sn_cell->priv->editable = g_object_ref (editable); sn_cell->priv->path = g_strdup (path); return GTK_CELL_EDITABLE (editable); } -static void -e_select_names_renderer_get_property (GObject *object, - guint property_id, - GValue *value, - GParamSpec *pspec) -{ - ESelectNamesRenderer *esnr = E_SELECT_NAMES_RENDERER (object); - - switch (property_id) { - case PROP_NAME: - g_value_set_string (value, esnr->priv->name); - break; - case PROP_EMAIL: - g_value_set_string (value, esnr->priv->email); - break; - default: - break; - } -} - -static void -e_select_names_renderer_set_property (GObject *object, - guint property_id, - const GValue *value, - GParamSpec *pspec) -{ - ESelectNamesRenderer *esnr = E_SELECT_NAMES_RENDERER (object); - - switch (property_id) { - case PROP_NAME: - g_free (esnr->priv->name); - esnr->priv->name = g_strdup (g_value_get_string (value)); - break; - case PROP_EMAIL: - g_free (esnr->priv->email); - esnr->priv->email = g_strdup (g_value_get_string (value)); - break; - default: - break; - } -} - -static void -e_select_names_renderer_finalize (GObject *object) -{ - ESelectNamesRendererPrivate *priv; - - priv = E_SELECT_NAMES_RENDERER_GET_PRIVATE (object); - - if (priv->editable != NULL) { - g_object_unref (priv->editable); - priv->editable = NULL; - } - - g_free (priv->path); - g_free (priv->name); - g_free (priv->email); - - /* Chain up to parent's finalize() method. */ - G_OBJECT_CLASS (e_select_names_renderer_parent_class)->finalize (object); -} - static void e_select_names_renderer_class_init (ESelectNamesRendererClass *class) { GObjectClass *object_class; - GtkCellRendererClass *cell_class = GTK_CELL_RENDERER_CLASS (class); + GtkCellRendererClass *renderer_class; g_type_class_add_private (class, sizeof (ESelectNamesRendererPrivate)); object_class = G_OBJECT_CLASS (class); - object_class->get_property = e_select_names_renderer_get_property; - object_class->set_property = e_select_names_renderer_set_property; - object_class->finalize = e_select_names_renderer_finalize; + object_class->get_property = select_names_renderer_get_property; + object_class->set_property = select_names_renderer_set_property; + object_class->dispose = select_names_renderer_dispose; + object_class->finalize = select_names_renderer_finalize; - cell_class->start_editing = e_select_names_renderer_start_editing; + renderer_class = GTK_CELL_RENDERER_CLASS (class); + renderer_class->start_editing = select_names_renderer_start_editing; g_object_class_install_property ( object_class, @@ -239,7 +258,8 @@ e_select_names_renderer_class_init (ESelectNamesRendererClass *class) "Name", "Email name.", NULL, - G_PARAM_READWRITE)); + G_PARAM_READWRITE | + G_PARAM_STATIC_STRINGS)); g_object_class_install_property ( object_class, @@ -249,7 +269,8 @@ e_select_names_renderer_class_init (ESelectNamesRendererClass *class) "Email", "Email address.", NULL, - G_PARAM_READWRITE)); + G_PARAM_READWRITE | + G_PARAM_STATIC_STRINGS)); signals[CELL_EDITED] = g_signal_new ( "cell_edited", @@ -265,9 +286,9 @@ e_select_names_renderer_class_init (ESelectNamesRendererClass *class) } static void -e_select_names_renderer_init (ESelectNamesRenderer *cell) +e_select_names_renderer_init (ESelectNamesRenderer *renderer) { - cell->priv = E_SELECT_NAMES_RENDERER_GET_PRIVATE (cell); + renderer->priv = E_SELECT_NAMES_RENDERER_GET_PRIVATE (renderer); } GtkCellRenderer * @@ -276,3 +297,43 @@ e_select_names_renderer_new (void) return g_object_new (E_TYPE_SELECT_NAMES_RENDERER, NULL); } +const gchar * +e_select_names_renderer_get_name (ESelectNamesRenderer *renderer) +{ + g_return_val_if_fail (E_IS_SELECT_NAMES_RENDERER (renderer), NULL); + + return renderer->priv->name; +} + +void +e_select_names_renderer_set_name (ESelectNamesRenderer *renderer, + const gchar *name) +{ + g_return_if_fail (E_IS_SELECT_NAMES_RENDERER (renderer)); + + g_free (renderer->priv->name); + renderer->priv->name = g_strdup (name); + + g_object_notify (G_OBJECT (renderer), "name"); +} + +const gchar * +e_select_names_renderer_get_email (ESelectNamesRenderer *renderer) +{ + g_return_val_if_fail (E_IS_SELECT_NAMES_RENDERER (renderer), NULL); + + return renderer->priv->email; +} + +void +e_select_names_renderer_set_email (ESelectNamesRenderer *renderer, + const gchar *email) +{ + g_return_if_fail (E_IS_SELECT_NAMES_RENDERER (renderer)); + + g_free (renderer->priv->email); + renderer->priv->email = g_strdup (email); + + g_object_notify (G_OBJECT (renderer), "email"); +} + diff --git a/calendar/gui/e-select-names-renderer.h b/calendar/gui/e-select-names-renderer.h index 2517f76d63..472a88fdd7 100644 --- a/calendar/gui/e-select-names-renderer.h +++ b/calendar/gui/e-select-names-renderer.h @@ -65,9 +65,20 @@ struct _ESelectNamesRendererClass { GList *names); }; -GType e_select_names_renderer_get_type (void) G_GNUC_CONST; +GType e_select_names_renderer_get_type + (void) G_GNUC_CONST; GtkCellRenderer * - e_select_names_renderer_new (void); + e_select_names_renderer_new (void); +const gchar * e_select_names_renderer_get_name + (ESelectNamesRenderer *renderer); +void e_select_names_renderer_set_name + (ESelectNamesRenderer *renderer, + const gchar *name); +const gchar * e_select_names_renderer_get_email + (ESelectNamesRenderer *renderer); +void e_select_names_renderer_set_email + (ESelectNamesRenderer *renderer, + const gchar *email); G_END_DECLS -- cgit v1.2.3