made restore from winspector.c automatically apply

ChangeLog

@@ -48,6 +48,10 @@ Changes since version 0.61.1:
 - added Jim Knoble <jmknoble@pobox.com>'s cursor thing patch
 - fixed lock of wmaker when clicking on menu multiple times
 - made transients appear near their owner
+- fixed crash bug with broken java implementations (Miguel Covarrubias 
+							<mcovarr@tigr.org>)
+- made Revert on attributes panel apply the reverted changes immediately
+  to avoid inconsistent internal state 
 
 Changes since version 0.61.0:
 .............................

WPrefs.app/MenuPreferences.c

@@ -204,7 +204,7 @@ createPanel(Panel *p)
     panel->wrapB = WMCreateSwitchButton(panel->optF);
     WMResizeWidget(panel->wrapB, 440, 32);
     WMMoveWidget(panel->wrapB, 25, 8);
-    WMSetButtonText(panel->wrapB, _("Always open submenus inside the screen, instead of scrolling.\nNote: this can be an annoyance at some circumstances."));
+    WMSetButtonText(panel->wrapB, _("Always open submenus inside the screen, instead of scrolling.\nNote: this is annoying."));
 
     panel->autoB = WMCreateSwitchButton(panel->optF);
     WMResizeWidget(panel->autoB, 440, 20);

WPrefs.app/Preferences.c

@@ -163,8 +163,8 @@ createPanel(Panel *p)
                             WMWidgetView(panel->sizeF));
 
     panel->sizeP = WMCreatePopUpButton(panel->sizeF);
-    WMResizeWidget(panel->sizeP, 180, 20);
+    WMResizeWidget(panel->sizeP, 200, 20);
-    WMMoveWidget(panel->sizeP, 32, 24);
+    WMMoveWidget(panel->sizeP, 22, 24);
     WMAddPopUpButtonItem(panel->sizeP, _("Corner of screen"));
     WMAddPopUpButtonItem(panel->sizeP, _("Center of screen"));
     WMAddPopUpButtonItem(panel->sizeP, _("Center of resized window"));
@@ -183,8 +183,8 @@ createPanel(Panel *p)
                             WMWidgetView(panel->posiF));
 
     panel->posiP = WMCreatePopUpButton(panel->posiF);
-    WMResizeWidget(panel->posiP, 180, 20);
+    WMResizeWidget(panel->posiP, 200, 20);
-    WMMoveWidget(panel->posiP, 32, 24);
+    WMMoveWidget(panel->posiP, 22, 24);
     WMAddPopUpButtonItem(panel->posiP, _("Corner of screen"));
     WMAddPopUpButtonItem(panel->posiP, _("Center of screen"));
     WMAddPopUpButtonItem(panel->posiP, _("Center of resized window"));

src/client.c

@@ -679,6 +679,13 @@ wClientGetNormalHints(WWindow *wwin, XWindowAttributes *wattribs, Bool geometry,
     }
 
     /* some buggy apps set weird hints.. */
+    if (wwin->normal_hints->min_width <= 0) 
+      wwin->normal_hints->min_width = MIN_WINDOW_SIZE;
+    
+    if (wwin->normal_hints->min_height <= 0)
+      wwin->normal_hints->min_height = MIN_WINDOW_SIZE;
+
+
     if (wwin->normal_hints->max_width < wwin->normal_hints->min_width)
 	wwin->normal_hints->max_width = wwin->normal_hints->min_width;
 
@@ -707,12 +714,6 @@ wClientGetNormalHints(WWindow *wwin, XWindowAttributes *wattribs, Bool geometry,
 	    wwin->normal_hints->max_aspect.y = 1;
     }
     
-    if (wwin->normal_hints->min_width <= 0) 
-      wwin->normal_hints->min_width = MIN_WINDOW_SIZE;
-    
-    if (wwin->normal_hints->min_height <= 0)
-      wwin->normal_hints->min_height = MIN_WINDOW_SIZE;
-
     if (wwin->normal_hints->min_height > wwin->normal_hints->max_height) {
 	wwin->normal_hints->min_height = wwin->normal_hints->max_height;
     }

src/window.c

@@ -1844,8 +1844,11 @@ wWindowConstrainSize(WWindow *wwin, int *nwidth, int *nheight)
 	height = (((height - minH) / hinc) * hinc) + minH;
     }
 
-    *nwidth = width;
+    /* broken stupid apps may cause preposterous values for these.. */
-    *nheight = height;
+    if (width > 0)
+	*nwidth = width;
+    if (height > 0)
+	*nheight = height;
 }
 
 

src/winspector.c

@@ -1045,6 +1045,11 @@ revertSettings(WMButton *button, InspectorPanel *panel)
     } else {
 	WMSetPopUpButtonSelectedItem(panel->wsP, 0);
     }
+    
+    /* must auto apply, so that there wno't be internal
+     * inconsistencies between the state in the flags and
+     * the actual state of the window */
+    applySettings(panel->applyBtn, panel);
 }