Re: [AD] X 3.9.28 report

[Peter Wang <tjaden@xxxxxxxxxx>]

Vincent PENQUERC'H <vpenquerch@xxxxxxxxxx> wrote:
> 
> Oh ... so sorry Michael :)

Well actually it turns out was a bug in Allegro :)  The extremely
simple fix is attached.

Michael: my man page says that hints->width and hints->height are
obsolete so I removed them, but perhaps you have a good reason to keep
them around?

Peter
-- 
tjaden@xxxxxxxxxx - http://www.psynet.net/~tjaden/
"The closer I get the worse it becomes..." - Nine Inch Nails
(Perhaps Trent is singing about a software project of his.)

--- xwin.c-3.9.28	Wed Nov 24 22:17:37 1999
+++ xwin.c	Wed Nov 24 22:16:01 1999
@@ -1878,11 +1878,6 @@
    if (_xwin.window == None)
       return;
 
-   /* Resize window.  */
-   _xwin.window_width = w;
-   _xwin.window_height = h;
-   XResizeWindow(_xwin.display, _xwin.window, w, h);
-
    /* Set size and position hints for Window Manager.  */
    hints = XAllocSizeHints();
    if (hints == 0)
@@ -1891,10 +1886,15 @@
    hints->flags = USSize | USPosition | PMinSize | PMaxSize | PBaseSize;
    hints->x = 0;
    hints->y = 0;
-   hints->width  = hints->min_width  = hints->max_width  = hints->base_width  = w;
-   hints->height = hints->min_height = hints->max_height = hints->base_height = h;
+   hints->min_width  = hints->max_width  = hints->base_width  = w;
+   hints->min_height = hints->max_height = hints->base_height = h;
    XSetWMNormalHints(_xwin.display, _xwin.window, hints);
    XFree(hints);
+
+   /* Resize window.  */
+   _xwin.window_width = w;
+   _xwin.window_height = h;
+   XResizeWindow(_xwin.display, _xwin.window, w, h);
 }