all repos — openbox @ 07fe0980bec54cb787579a6b2633309d20fc083b

openbox fork - make it a bit more like ryudo 

declare variables at the beginning of blocks, because it is nice to do so. and polite?
Dana Jansens danakj@orodu.net
commit

07fe0980bec54cb787579a6b2633309d20fc083b

parent

de383f35f792e05259da42263fb8b1dba7d2a3f5

2 files changed, 6 insertions(+), 5 deletions(-)

jump to
M openbox/menuframe.c → openbox/menuframe.c 
@@ -676,13 +676,14 @@ }
 
 void menu_frame_hide_all()
 {
+    GList *it;
+
     if (config_submenu_show_delay) {
         /* remove any submenu open requests */
         ob_main_loop_timeout_remove(ob_main_loop,
                                     menu_entry_frame_submenu_timeout);
     }
-    GList *it = g_list_last(menu_frame_visible);
-    if (it) 
+    if ((it = g_list_last(menu_frame_visible)))
         menu_frame_hide(it->data);
 }
M render/gradient.c → render/gradient.c 
@@ -371,6 +371,9 @@ RrPixel32 *data = sf->pixel_data;
     RrPixel32 current;
     RrColor *primary_light, *secondary_light;
 
+    VARS(y1);
+    VARS(y3);
+
     r = sf->primary->r;
     r += r >> 2;
     g = sf->primary->g;

@@ -393,10 +396,7 @@ if (g > 0xFF) g = 0xFF;
     if (b > 0xFF) b = 0xFF;
     secondary_light = RrColorNew(a->inst, r, g, b);
 
-    VARS(y1);
     SETUP(y1, primary_light, sf->primary, (h / 2) -1);
-  
-    VARS(y3);
     SETUP(y3, sf->secondary, secondary_light,  (h / 2) -1);
 
     for (y1 = h - 1; y1 > (h / 2) -1; --y1) {  /* 0 -> h-1 */