After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 572241 - Remove deprecated GTK+ symbols
Remove deprecated GTK+ symbols
Status: RESOLVED FIXED
Product: libegg
Classification: Other
Component: other
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Libegg maintenance
Libegg maintenance
Depends on:
Blocks: 585692
 
 
Reported: 2009-02-18 04:37 UTC by André Klapper
Modified: 2009-06-13 23:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description André Klapper 2009-02-18 04:37:56 UTC 
http://live.gnome.org/GnomeGoals/RemoveDeprecatedSymbols/GTK%2B

./src/eggfindbar.c:
 gtk_tool_item_set_tooltip (priv->previous_button, GTK_TOOLBAR (find_bar)->tooltips,
./src/eggfindbar.c:
 gtk_tool_item_set_tooltip (priv->next_button, GTK_TOOLBAR (find_bar)->tooltips,
./src/eggfindbar.c:
 gtk_tool_item_set_tooltip (priv->case_button, GTK_TOOLBAR (find_bar)->tooltips,


http://library.gnome.org/devel/gtk/2.11/GtkToolItem.html#gtk-tool-item-set-tooltip
Comment 1 Richard Hult 2009-02-19 10:21:57 UTC 
This needs fixing in libegg, we are you slurping that in...
Comment 2 Richard Hult 2009-02-19 10:22:07 UTC 
s/you/just/ :)
Comment 3 André Klapper 2009-02-19 10:54:50 UTC 
Alright, let's move :)
Comment 4 Felix Riemann 2009-02-23 10:50:36 UTC 
Looks like it's already fixed but just ifdeffed:


360 #if GTK_CHECK_VERSION (2, 11, 5)
361 	gtk_widget_set_tooltip_text (GTK_WIDGET (priv->case_button),
362 	_("Toggle case sensitive search"));
363 #else
364 	gtk_tool_item_set_tooltip (priv->case_button, GTK_TOOLBAR (find_bar)->tooltips,
365 	_("Toggle case sensitive search"),
366 	NULL);
367 #endif


Btw, there's no eggfindbar in libegg. Google indicates that it is developed in evince's source tree.
Comment 5 Vincent Untz 2009-02-26 05:00:37 UTC 
Yeah, so:
 + not in libegg
 + already fixed

Really closing the bug :-)
Comment 6 André Klapper 2009-04-26 16:37:30 UTC 
OK, Comment to myself:
Currently http://www.gnome.org/~fpeters/299.html lists for devhelp **2.22**
"GTK_CHECK_CAST, GTK_CHECK_CLASS_CAST, GTK_CHECK_CLASS_TYPE, GTK_CHECK_TYPE, GtkType, gtk_box_pack_end_defaults"
as deprecated GTK+ symbols.
All these deprecated GTK+ symbols do not exist anymore in devhelp trunk.
Hence whitelisting them.