GNOME Bugzilla – Bug 589769
Remove deprecated Glib and GTK+ symbols
Last modified: 2009-10-08 19:15:35 UTC
According to http://www.gnome.org/~fpeters/299.html gegl uses deprecated Glib and GTK symbols. These symbols will be removed for GTK 3. * g_win32_get_package_installation_subdirectory * GtkSignalFunc * gtk_signal_connect $:andre\> grep -r gtk_signal_connect . ./examples/gegl-paint.c: gtk_signal_connect (GTK_OBJECT (view), "motion-notify-event", ./examples/gegl-paint.c: gtk_signal_connect (GTK_OBJECT (view), "button-press-event", ./examples/gegl-paint.c: gtk_signal_connect (GTK_OBJECT (view), "button-release-event", $:andre\> grep -r GtkSignalFunc . ./examples/gegl-paint.c: (GtkSignalFunc) paint_motion, NULL); ./examples/gegl-paint.c: (GtkSignalFunc) paint_press, NULL); ./examples/gegl-paint.c: (GtkSignalFunc) paint_release, NULL); $:andre\> grep -r g_win32_get_package_installation_subdirectory . ./gegl/gegl-init.c: g_win32_get_package_installation_subdirectory (NULL,
Let's look at this for the next release. A patch (or rather, git commit) that fixes it would be appreciated.
Sorry, I myself don't code - I've set the gnome-love keyword here. For API refs see http://library.gnome.org/devel/gtk/unstable/gtk-Signals.html#gtk-signal-connect and http://library.gnome.org/devel/glib/stable/glib-Windows-Compatibility-Functions.html#g-win32-get-package-installation-subdirectory . And I assume it's just a simple "GtkSignalFunc" -> "GCallback"...
g_win32_get_package_installation_subdirectory is gone.
Created attachment 141130 [details] [review] Remove deprecated GTK+ symbols This patch removes the remaining GTK+ symbols
Hello, Could some GEGL developer review the latest patch (and we can close this bug ;))?
Thanks for the patch, looks fine to me, pushed commit ccb3a25eb984191ba0236f3a15902c8bcd1a82f0 Author: Javier Jardón <javierjc1982@gmail.com> Date: Wed Aug 19 04:43:34 2009 +0200 Bug 589769 - Remove deprecated Glib and GTK+ symbols examples/gegl-paint.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-)