diff options
-rw-r--r-- | ChangeLog | 20 | ||||
-rw-r--r-- | embed/ephy-embed-event.c | 7 | ||||
-rw-r--r-- | embed/ephy-embed-event.h | 11 | ||||
-rw-r--r-- | embed/mozilla/mozilla-embed-event.cpp | 7 | ||||
-rw-r--r-- | src/ephy-tab.c | 4 | ||||
-rw-r--r-- | src/ephy-window.c | 8 | ||||
-rw-r--r-- | src/epiphany.defs | 3 | ||||
-rw-r--r-- | src/epiphany.override | 2 | ||||
-rw-r--r-- | src/popup-commands.c | 28 |
9 files changed, 52 insertions, 38 deletions
@@ -1,3 +1,23 @@ +2006-01-17 Crispin Flowerday <gnome@flowerday.cx> + + * embed/ephy-embed-event.c: (ephy_embed_event_get_property): + * embed/ephy-embed-event.h: + * embed/mozilla/mozilla-embed-event.cpp: + * src/ephy-tab.c: (save_property_url), + (ephy_tab_dom_mouse_click_cb): + * src/ephy-window.c: (update_popups_tooltips), (show_embed_popup): + * src/epiphany.defs: + * src/epiphany.override: + * src/popup-commands.c: (popup_cmd_link_in_new_window), + (popup_cmd_link_in_new_tab), (popup_cmd_bookmark_link), + (popup_cmd_copy_link_address), (save_property_url), + (popup_cmd_open_link), (popup_cmd_set_image_as_background), + (popup_cmd_copy_image_location), (popup_cmd_open_image): + + Change the 'ephy_embed_event_get_property' function + return a GValue* rather than taking a pointer to it as + an argument. + 2006-01-17 Christian Persch <chpe@cvs.gnome.org> * data/glade/prefs-dialog.glade: diff --git a/embed/ephy-embed-event.c b/embed/ephy-embed-event.c index 7a4ef6826..069a839d7 100644 --- a/embed/ephy-embed-event.c +++ b/embed/ephy-embed-event.c @@ -88,13 +88,12 @@ ephy_embed_event_get_coords (EphyEmbedEvent *event, iface->get_coordinates (event, x, y); } -void +const GValue* ephy_embed_event_get_property (EphyEmbedEvent *event, - const char *name, - const GValue **value) + const char *name) { EphyEmbedEventIface *iface = EPHY_EMBED_EVENT_GET_IFACE (event); - iface->get_property (event, name, value); + return iface->get_property (event, name); } gboolean diff --git a/embed/ephy-embed-event.h b/embed/ephy-embed-event.h index 95d59d073..075a971b0 100644 --- a/embed/ephy-embed-event.h +++ b/embed/ephy-embed-event.h @@ -61,9 +61,8 @@ struct _EphyEmbedEventIface void (* get_coordinates) (EphyEmbedEvent *event, guint *x, guint *y); - void (* get_property) (EphyEmbedEvent *event, - const char *name, - const GValue **value); + const GValue* (* get_property) (EphyEmbedEvent *event, + const char *name); gboolean (* has_property) (EphyEmbedEvent *event, const char *name); gpointer (* get_dom_event) (EphyEmbedEvent *event); @@ -85,10 +84,8 @@ guint ephy_embed_event_get_modifier (EphyEmbedEvent *event); void ephy_embed_event_get_coords (EphyEmbedEvent *event, guint *x, guint *y); - -void ephy_embed_event_get_property (EphyEmbedEvent *event, - const char *name, - const GValue **value); +const GValue* ephy_embed_event_get_property (EphyEmbedEvent *event, + const char *name); gboolean ephy_embed_event_has_property (EphyEmbedEvent *event, const char *name); diff --git a/embed/mozilla/mozilla-embed-event.cpp b/embed/mozilla/mozilla-embed-event.cpp index cbd2802d4..eae87d95c 100644 --- a/embed/mozilla/mozilla-embed-event.cpp +++ b/embed/mozilla/mozilla-embed-event.cpp @@ -140,12 +140,11 @@ impl_get_coordinates (EphyEmbedEvent *event, *y = ((MozillaEmbedEvent *) event)->y; } -static void +static const GValue* impl_get_property (EphyEmbedEvent *event, - const char *name, - const GValue **value) + const char *name) { - *value = (const GValue *) g_hash_table_lookup (((MozillaEmbedEvent *) event)->priv->props, name); + return (const GValue *) g_hash_table_lookup (((MozillaEmbedEvent *) event)->priv->props, name); } static gboolean diff --git a/src/ephy-tab.c b/src/ephy-tab.c index 3d7a5dce0..838e192bf 100644 --- a/src/ephy-tab.c +++ b/src/ephy-tab.c @@ -1870,7 +1870,7 @@ save_property_url (EphyEmbed *embed, const GValue *value; EphyEmbedPersist *persist; - ephy_embed_event_get_property (event, property, &value); + value = ephy_embed_event_get_property (event, property); location = g_value_get_string (value); persist = EPHY_EMBED_PERSIST @@ -1951,7 +1951,7 @@ ephy_tab_dom_mouse_click_cb (EphyEmbed *embed, const GValue *value; const char *link_address; - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); link_address = g_value_get_string (value); handled = open_link_in_new_tab (tab, link_address); } diff --git a/src/ephy-window.c b/src/ephy-window.c index 1c4178f18..7e9c965c2 100644 --- a/src/ephy-window.c +++ b/src/ephy-window.c @@ -1835,7 +1835,7 @@ update_popups_tooltips (EphyWindow *window, EphyEmbedEvent *event) if (context & EPHY_EMBED_CONTEXT_IMAGE) { - ephy_embed_event_get_property (event, "image", &value); + value = ephy_embed_event_get_property (event, "image"); name = get_name_from_address_value (value); action = gtk_action_group_get_action (group, "OpenImage"); @@ -1864,7 +1864,7 @@ update_popups_tooltips (EphyWindow *window, EphyEmbedEvent *event) if (context & EPHY_EMBED_CONTEXT_EMAIL_LINK) { - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); action = gtk_action_group_get_action (group, "SendEmail"); tooltip = g_strdup_printf (_("Send email to address ā%sā"), @@ -1881,7 +1881,7 @@ update_popups_tooltips (EphyWindow *window, EphyEmbedEvent *event) if (context & EPHY_EMBED_CONTEXT_LINK) { - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); action = gtk_action_group_get_action (group, "DownloadLink"); name = get_name_from_address_value (value); @@ -1925,7 +1925,7 @@ show_embed_popup (EphyWindow *window, return; } - ephy_embed_event_get_property (event, "framed_page", &value); + value = ephy_embed_event_get_property (event, "framed_page"); framed = g_value_get_int (value); can_open_in_new = ephy_embed_event_has_property (event, "link-has-web-scheme"); diff --git a/src/epiphany.defs b/src/epiphany.defs index 78b910b78..15e5bcb76 100644 --- a/src/epiphany.defs +++ b/src/epiphany.defs @@ -599,10 +599,9 @@ (define-method get_property (of-object "EphyEmbedEvent") (c-name "ephy_embed_event_get_property") - (return-type "none") + (return-type "const-GValue*") (parameters '("const-char*" "name") - '("const-GValue**" "value") ) ) diff --git a/src/epiphany.override b/src/epiphany.override index 5018bd9f2..1d1535f79 100644 --- a/src/epiphany.override +++ b/src/epiphany.override @@ -359,7 +359,7 @@ _wrap_ephy_embed_event_get_property (PyGObject *self, "s:EphyEmbedEvent.get_property", kwlist, &prop)) return NULL; - ephy_embed_event_get_property (EPHY_EMBED_EVENT (self->obj), prop, &value); + value = ephy_embed_event_get_property (EPHY_EMBED_EVENT (self->obj), prop); return pyg_value_as_pyobject(value, TRUE); } diff --git a/src/popup-commands.c b/src/popup-commands.c index f05167bb4..107304f83 100644 --- a/src/popup-commands.c +++ b/src/popup-commands.c @@ -51,7 +51,7 @@ popup_cmd_link_in_new_window (GtkAction *action, event = ephy_window_get_context_event (window); g_return_if_fail (event != NULL); - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); ephy_shell_new_tab (ephy_shell, NULL, tab, g_value_get_string (value), @@ -72,7 +72,7 @@ popup_cmd_link_in_new_tab (GtkAction *action, event = ephy_window_get_context_event (window); g_return_if_fail (event != NULL); - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); ephy_shell_new_tab (ephy_shell, window, tab, g_value_get_string (value), @@ -98,11 +98,11 @@ popup_cmd_bookmark_link (GtkAction *action, event = ephy_window_get_context_event (window); g_return_if_fail (event != NULL); - ephy_embed_event_get_property (event, "link_is_smart", &link_is_smart); - ephy_embed_event_get_property (event, "link", &link); - ephy_embed_event_get_property (event, "link_title", &link_title); - ephy_embed_event_get_property (event, "link_rel", &link_rel); - ephy_embed_event_get_property (event, "linktext", &linktext); + link_is_smart = ephy_embed_event_get_property (event, "link_is_smart"); + link = ephy_embed_event_get_property (event, "link"); + link_title = ephy_embed_event_get_property (event, "link_title"); + link_rel = ephy_embed_event_get_property (event, "link_rel"); + linktext = ephy_embed_event_get_property (event, "linktext"); location = g_value_get_string (link); g_return_if_fail (location); @@ -155,13 +155,13 @@ popup_cmd_copy_link_address (GtkAction *action, if (context & EPHY_EMBED_CONTEXT_EMAIL_LINK) { - ephy_embed_event_get_property (event, "email", &value); + value = ephy_embed_event_get_property (event, "email"); address = g_value_get_string (value); popup_cmd_copy_to_clipboard (window, address); } else if (context & EPHY_EMBED_CONTEXT_LINK) { - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); address = g_value_get_string (value); popup_cmd_copy_to_clipboard (window, address); } @@ -186,7 +186,7 @@ save_property_url (GtkAction *action, embed = ephy_window_get_active_embed (window); g_return_if_fail (embed != NULL); - ephy_embed_event_get_property (event, property, &value); + value = ephy_embed_event_get_property (event, property); location = g_value_get_string (value); persist = EPHY_EMBED_PERSIST @@ -219,7 +219,7 @@ popup_cmd_open_link (GtkAction *action, g_return_if_fail (embed != NULL); event = ephy_window_get_context_event (window); - ephy_embed_event_get_property (event, "link", &value); + value = ephy_embed_event_get_property (event, "link"); location = g_value_get_string (value); ephy_embed_load_url (embed, location); @@ -292,7 +292,7 @@ popup_cmd_set_image_as_background (GtkAction *action, embed = ephy_window_get_active_embed (window); g_return_if_fail (embed != NULL); - ephy_embed_event_get_property (event, "image", &value); + value = ephy_embed_event_get_property (event, "image"); location = g_value_get_string (value); persist = EPHY_EMBED_PERSIST @@ -327,7 +327,7 @@ popup_cmd_copy_image_location (GtkAction *action, const GValue *value; event = ephy_window_get_context_event (window); - ephy_embed_event_get_property (event, "image", &value); + value = ephy_embed_event_get_property (event, "image"); location = g_value_get_string (value); popup_cmd_copy_to_clipboard (window, location); } @@ -435,7 +435,7 @@ popup_cmd_open_image (GtkAction *action, embed = ephy_window_get_active_embed (window); g_return_if_fail (embed != NULL); - ephy_embed_event_get_property (event, "image", &value); + value = ephy_embed_event_get_property (event, "image"); address = g_value_get_string (value); scheme = gnome_vfs_get_uri_scheme (address); |