]> Pileus Git - ~andy/gtk/commitdiff
Don't return prematurely
authorMatthias Clasen <mclasen@redhat.com>
Fri, 30 Nov 2012 04:05:07 +0000 (23:05 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Fri, 30 Nov 2012 04:10:35 +0000 (23:10 -0500)
This code was clearly meant to return _after_ the loop.
Don't put the return inside the loop body, then.

gtk/gtkcssshorthandpropertyimpl.c

index 0b467ab549ef0837bf9ad675f182667eb0319564..b01494b7da888fd165f144482b339b94423f49d6 100644 (file)
@@ -565,8 +565,8 @@ parse_background (GtkCssShorthandProperty  *shorthand,
           {
             g_ptr_array_set_free_func (arrays[i], (GDestroyNotify) _gtk_css_value_unref);
             g_ptr_array_unref (arrays[i]);
-            return FALSE;
           }
+        return FALSE;
       }
 
       for (i = 0; i < 6; i++)
@@ -668,8 +668,8 @@ parse_transition (GtkCssShorthandProperty  *shorthand,
           {
             g_ptr_array_set_free_func (arrays[i], (GDestroyNotify) _gtk_css_value_unref);
             g_ptr_array_unref (arrays[i]);
-            return FALSE;
           }
+        return FALSE;
       }
 
       for (i = 0; i < 4; i++)
@@ -784,8 +784,8 @@ parse_animation (GtkCssShorthandProperty  *shorthand,
           {
             g_ptr_array_set_free_func (arrays[i], (GDestroyNotify) _gtk_css_value_unref);
             g_ptr_array_unref (arrays[i]);
-            return FALSE;
           }
+        return FALSE;
       }
 
       for (i = 0; i < 6; i++)