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 596494 - New property "cursor" in 2.18's GdkWindow with wrong type?
New property "cursor" in 2.18's GdkWindow with wrong type?
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Backend: X11
2.18.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2009-09-27 07:58 UTC by Guillaume Cottenceau
Modified: 2009-09-27 19:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Boxed paramspec (1008 bytes, patch)
2009-09-27 18:03 UTC, Cody Russell
committed Details | Review

Description Guillaume Cottenceau 2009-09-27 07:58:06 UTC
New property "cursor" in 2.18's GdkWindow has type "gpointer":

http://library.gnome.org/devel/gdk/2.18/gdk-Windows.html#gdk-Windows.properties

This leads to a crash in the ruby-gnome2 bindings when manipulating GdkWindow's cursor with the 2.18 runtime, seemingly because the type should be more precise, as we can see for example in GtkWidget:

http://library.gnome.org/devel/gtk/2.18/GtkWidget.html#GtkWidget.properties

e.g. why cursor property of GdkWindow doesn't have type "GdkCursor*"?

Thanks.
Comment 1 Cody Russell 2009-09-27 18:03:36 UTC
Created attachment 144127 [details] [review]
Boxed paramspec
Comment 2 Christian Dywan 2009-09-27 19:31:22 UTC
I'm not sure why Ruby would crash, but I think it's obviously a good idea to use the type. It's API compatible as far as I realize since it's still a pointer type and there's no way to pass a different kind of pointer here.
Comment 3 Cody Russell 2009-09-27 19:39:22 UTC
Comment on attachment 144127 [details] [review]
Boxed paramspec

Approved by mclasen on irc