winex11.drv: Don't put the GL drawable offscreen.

Matteo Bruni mbruni at codeweavers.com
Tue Jan 24 17:08:01 CST 2012


There is no need to create the window offscreen anymore as it is immediately
redirected offscreen anyway and it has no real parent.

Thanks to Lauri Koponen for noticing this piece of code in bug 29630.
---
 dlls/winex11.drv/window.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c
index 48ba000..fcf5e0e 100644
--- a/dlls/winex11.drv/window.c
+++ b/dlls/winex11.drv/window.c
@@ -591,7 +591,7 @@ static BOOL set_win_format( HWND hwnd, XID fbconfig_id )
         XInstallColormap(gdi_display, attrib.colormap);
 
         if(data->gl_drawable) XDestroyWindow(gdi_display, data->gl_drawable);
-        data->gl_drawable = XCreateWindow(gdi_display, dummy_parent, -w, 0, w, h, 0,
+        data->gl_drawable = XCreateWindow(gdi_display, dummy_parent, 0, 0, w, h, 0,
                                           vis->depth, InputOutput, vis->visual,
                                           CWColormap | CWOverrideRedirect,
                                           &attrib);
@@ -669,7 +669,7 @@ static void sync_gl_drawable(struct x11drv_win_data *data)
     if(usexcomposite)
     {
         wine_tsx11_lock();
-        XMoveResizeWindow(gdi_display, data->gl_drawable, -w, 0, w, h);
+        XMoveResizeWindow(gdi_display, data->gl_drawable, 0, 0, w, h);
         wine_tsx11_unlock();
         return;
     }
-- 
1.7.3.4




More information about the wine-patches mailing list