[patch] color picker crash
- From: Erwann Chenede <Erwann Chenede Sun COM>
- To: gtk-devel-list gnome org
- Subject: [patch] color picker crash
- Date: Mon, 29 Apr 2002 18:13:26 +0100 (BST)
Hi,
I just found a bug in color picker (grab release problem).
Here is a patch to fix it.
Index: gdk/x11/gdkimage-x11.c
===================================================================
RCS file: /cvs/gnome/gtk+/gdk/x11/gdkimage-x11.c,v
retrieving revision 1.43
diff -u -r1.43 gdkimage-x11.c
--- gdk/x11/gdkimage-x11.c 25 Apr 2002 22:29:12 -0000 1.43
+++ gdk/x11/gdkimage-x11.c 29 Apr 2002 16:41:07 -0000
@@ -652,7 +652,7 @@
if (have_grab)
{
gdk_x11_display_ungrab (gdk_drawable_get_display (drawable));
- XFlush (GDK_DRAWABLE_XDISPLAY (drawable));
+ XFlush (gdk_x11_drawable_get_xdisplay (drawable));
have_grab = FALSE;
}
The problem is that GDK_DRAWABLE_XDISPLAY expect a GdkDrawable as oppose
to gdk_x11_drawable_get_xdisplay which can deal with GdkDrawable and
GdkDrawableImplX11.
So I guess an other patch would be the change GDK_DRAWABLE_XDISPLAY
itself to be gdk_x11_drawable_get_xdisplay.
What is best ?
If the patch is ok, can I commit ?
Best Regards,
Erwann
Erwann Chénedé, Sun Microsystems, Ireland
Phone : +353 1 8199031 xt: 19031
[ I speak for myself, not for my employer ]
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]