M openbox/client.c → openbox/client.c

@@ -1980,7 +1980,7 @@ 
     if (!got &&
         PROP_GETA32(self->window, net_wm_strut, cardinal, &data, &num)) {
         if (num == 4) {
-            const Rect *a;
+            Rect *a;
 
             got = TRUE;
 

M openbox/menuframe.c → openbox/menuframe.c

@@ -952,7 +952,7 @@ return FALSE;
 
     /* find the monitor the menu is on */
     for (i = 0; i < screen_num_monitors; ++i) {
-        Rect a = screen_physical_area_monitor(i);
+        Rect *a = screen_physical_area_monitor(i);
         gboolean contains = RECT_CONTAINS(*a, x, y);
         g_free(a);
         if (contains) {

M openbox/place.c → openbox/place.c

@@ -112,15 +112,16 @@ }
 
     screen_pointer_pos(&px, &py);
 
-    for (i = 0; i < screen_num_monitors; i++)
+    for (i = 0; i < screen_num_monitors; i++) {
         Rect *monitor = screen_physical_area_monitor(i);
         gboolean contain = RECT_CONTAINS(*monitor, px, py);
         g_free(monitor);
-        if (contain)
+        if (contain) {
             add_choice(choice, i);
             ob_debug("placement adding choice %d for mouse pointer\n", i);
             break;
         }
+    }
 
     /* add any leftover choices */
     for (i = 0; i < screen_num_monitors; ++i)