WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 20078
[GTK] Use G_* macros instead of deprecated GTK_* macros in gtk2xtbin.h
https://bugs.webkit.org/show_bug.cgi?id=20078
Summary
[GTK] Use G_* macros instead of deprecated GTK_* macros in gtk2xtbin.h
Wouter Bolsterlee
Reported
2008-07-17 06:58:52 PDT
The GTK type checking macros have been deprecated in favor for the (identical) glib ones. Using the old ones breaks the build with -DG_DISABLE_DEPRECATED -DGTK_DISABLE_DEPRECATED enabled. Will attach a patch in a minute.
Attachments
Proposed fix
(1.18 KB, patch)
2008-07-17 07:03 PDT
,
Wouter Bolsterlee
zecke
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Wouter Bolsterlee
Comment 1
2008-07-17 07:03:57 PDT
Created
attachment 22338
[details]
Proposed fix
Holger Freyther
Comment 2
2008-07-17 09:58:42 PDT
Comment on
attachment 22338
[details]
Proposed fix Compared with
http://library.gnome.org/devel/gtk/stable/gtk-Types.html#GTK-CHECK-CAST:CAPS
. I couldn't see a deprecation note though.
Wouter Bolsterlee
Comment 3
2008-07-17 12:18:56 PDT
It will be deprecated in the upcoming GTK release [1]. The corresponding Glib are also in the current stable release [2]. [1]
http://library.gnome.org/devel/gtk/unstable/gtk-Types.html#GTK-CHECK-CAST:CAPS
(note that there's "unstable", not "stable" in the url!) [2]
http://library.gnome.org/devel/gobject/stable/gobject-Type-Information.html#G-TYPE-CHECK-CLASS-CAST:CAPS
Wouter Bolsterlee
Comment 4
2008-07-17 12:22:04 PDT
Note that the GTK checks macros are exactly the same as the Glib macros, as can be seen in gtk+/gtk/gtktypeutils.h:
> /* glib macro wrappers (compatibility) */ > #define GTK_CHECK_CAST G_TYPE_CHECK_INSTANCE_CAST > #define GTK_CHECK_CLASS_CAST G_TYPE_CHECK_CLASS_CAST > #define GTK_CHECK_GET_CLASS G_TYPE_INSTANCE_GET_CLASS > #define GTK_CHECK_TYPE G_TYPE_CHECK_INSTANCE_TYPE > #define GTK_CHECK_CLASS_TYPE G_TYPE_CHECK_CLASS_TYPE
Jan Alonzo
Comment 5
2008-07-25 16:18:29 PDT
landed in
r35361
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug