all repos — openbox @ 1387b33394a219e0a64015d614fc9a44e611901b

openbox fork - make it a bit more like ryudo 

Fix an if statement getting moved into a loop it shouldn't have

Found by coverity code analysis.
Mikael Magnusson mikachu@gmail.com
commit

1387b33394a219e0a64015d614fc9a44e611901b

parent

396198081e72d6d758d51d4c6cdda1105be0df2d

1 files changed, 5 insertions(+), 5 deletions(-)

jump to
M openbox/place.c → openbox/place.c 
@@ -360,11 +360,11 @@ r = MAX(r, RECT_RIGHT(m->frame->area));
                     b = MAX(b, RECT_BOTTOM(m->frame->area));
                 }
             }
-            if (!first) {
-                *x = ((r + 1 - l) - frame_size.width) / 2 + l;
-                *y = ((b + 1 - t) - frame_size.height) / 2 + t;
-                return TRUE;
-            }
+        }
+        if (!first) {
+            *x = ((r + 1 - l) - frame_size.width) / 2 + l;
+            *y = ((b + 1 - t) - frame_size.height) / 2 + t;
+            return TRUE;
         }
     }