[gtk+/gtk-style-context: 91/533] GtkCssProvider: Make additions to parse_value() less error prone.



commit fede1275979a910581cff8e3325e61c093ba6c62
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri Jun 18 16:42:48 2010 +0200

    GtkCssProvider: Make additions to parse_value() less error prone.

 gtk/gtkcssprovider.c |   23 +++++++++--------------
 1 files changed, 9 insertions(+), 14 deletions(-)
---
diff --git a/gtk/gtkcssprovider.c b/gtk/gtkcssprovider.c
index e61ea73..acc73c0 100644
--- a/gtk/gtkcssprovider.c
+++ b/gtk/gtkcssprovider.c
@@ -874,6 +874,8 @@ parse_value (GType        type,
              const gchar *value_str,
              GValue      *value)
 {
+  gboolean parsed = TRUE;
+
   g_value_init (value, type);
 
   if (type == GDK_TYPE_COLOR)
@@ -884,7 +886,6 @@ parse_value (GType        type,
         return FALSE;
 
       g_value_set_boxed (value, &color);
-      return TRUE;
     }
   else if (type == PANGO_TYPE_FONT_DESCRIPTION)
     {
@@ -892,31 +893,25 @@ parse_value (GType        type,
 
       font_desc = pango_font_description_from_string (value_str);
       g_value_take_boxed (value, font_desc);
-
-      return TRUE;
     }
   else if (type == G_TYPE_INT)
-    {
-      g_value_set_int (value, atoi (value_str));
-      return TRUE;
-    }
+    g_value_set_int (value, atoi (value_str));
   else if (type == G_TYPE_DOUBLE)
-    {
-      g_value_set_double (value, g_ascii_strtod (value_str, NULL));
-      return TRUE;
-    }
+    g_value_set_double (value, g_ascii_strtod (value_str, NULL));
   else if (type == GTK_TYPE_THEMING_ENGINE)
     {
       GtkThemingEngine *engine;
 
       engine = gtk_theming_engine_load (value_str);
       g_value_set_object (value, engine);
-      return TRUE;
     }
   else
-    g_warning ("Cannot parse string '%s' for type %s", value_str, g_type_name (type));
+    {
+      g_warning ("Cannot parse string '%s' for type %s", value_str, g_type_name (type));
+      parsed = FALSE;
+    }
 
-  return FALSE;
+  return parsed;
 }
 
 static GTokenType



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]