[gtk+/parser: 57/57] cssprovider: Move properties to scanner constructor



commit 66b92ac4fefafc80374507ef615abd4d0ca09e59
Author: Benjamin Otte <otte redhat com>
Date:   Mon Apr 11 16:48:00 2011 +0200

    cssprovider: Move properties to scanner constructor
    
    So we can set them in one place.

 gtk/gtkcssprovider.c |   44 +++++++++++++++++++++++++++-----------------
 1 files changed, 27 insertions(+), 17 deletions(-)
---
diff --git a/gtk/gtkcssprovider.c b/gtk/gtkcssprovider.c
index 390db3a..000b416 100644
--- a/gtk/gtkcssprovider.c
+++ b/gtk/gtkcssprovider.c
@@ -788,6 +788,7 @@ struct SelectorStyleInfo
 
 struct _GtkCssScannerPrivate
 {
+  GFile *file;
   GSList *state;
   GSList *cur_selectors;
   GHashTable *cur_properties;
@@ -1137,6 +1138,8 @@ gtk_css_scanner_destroy (GScanner *scanner)
 
   gtk_css_scanner_reset (scanner);
 
+  if (priv->file)
+    g_object_unref (priv->file);
   g_hash_table_destroy (priv->cur_properties);
   g_slice_free (GtkCssScannerPrivate, priv);
   
@@ -1144,7 +1147,9 @@ gtk_css_scanner_destroy (GScanner *scanner)
 }
 
 static GScanner *
-gtk_css_scanner_new (void)
+gtk_css_scanner_new (GFile       *file,
+                     const gchar *data,
+                     gsize        length)
 {
   GtkCssScannerPrivate *priv;
   GScanner *scanner;
@@ -1153,6 +1158,9 @@ gtk_css_scanner_new (void)
 
   priv = scanner->user_data = g_slice_new0 (GtkCssScannerPrivate);
 
+  if (file)
+    priv->file = g_object_ref (file);
+
   priv->cur_properties = g_hash_table_new_full (g_str_hash,
                                                 g_str_equal,
                                                 (GDestroyNotify) g_free,
@@ -1179,6 +1187,11 @@ gtk_css_scanner_new (void)
 
   scanner_apply_scope (scanner, SCOPE_SELECTOR);
 
+  if (length > G_MAXUINT32)
+    g_warning ("CSS file too large, truncating");
+
+  g_scanner_input_text (scanner, data, length);
+
   return scanner;
 }
 
@@ -1603,12 +1616,16 @@ gtk_css_provider_take_error (GtkCssProvider *provider,
                              GScanner       *scanner,
                              GError         *error)
 {
-  const char *filename;
+  char *filename;
   guint line, position;
 
   if (scanner)
     {
-      filename = scanner->input_name;
+      GtkCssScannerPrivate *priv = scanner->user_data;
+      if (priv->file)
+        filename = g_file_get_path (priv->file);
+      else
+        filename = NULL;
       line = scanner->line;
       position = scanner->position;
     }
@@ -1622,6 +1639,7 @@ gtk_css_provider_take_error (GtkCssProvider *provider,
   g_signal_emit (provider, css_provider_signals[PARSING_ERROR], 0,
                  filename, line, position, error);
 
+  g_free (filename);
   g_error_free (error);
 }
 
@@ -2661,10 +2679,7 @@ gtk_css_provider_load_from_data (GtkCssProvider  *css_provider,
 
   gtk_css_provider_reset (css_provider);
 
-  scanner = gtk_css_scanner_new ();
-
-  scanner->input_name = NULL;
-  g_scanner_input_text (scanner, data, (guint) length);
+  scanner = gtk_css_scanner_new (NULL, data, length);
 
   result = parse_stylesheet (css_provider, scanner, error);
 
@@ -2692,7 +2707,6 @@ gtk_css_provider_load_from_file (GtkCssProvider  *css_provider,
   GtkCssProviderPrivate *priv;
   GError *internal_error = NULL;
   GScanner *scanner;
-  char *path;
   gchar *data;
   gsize length;
   gboolean ret;
@@ -2712,15 +2726,10 @@ gtk_css_provider_load_from_file (GtkCssProvider  *css_provider,
 
   gtk_css_provider_reset (css_provider);
 
-  scanner = gtk_css_scanner_new ();
-
-  path = g_file_get_path (file);
-  scanner->input_name = path;
-  g_scanner_input_text (scanner, data, (guint) length);
+  scanner = gtk_css_scanner_new (file, data, length);
 
   ret = parse_stylesheet (css_provider, scanner, error);
 
-  g_free (path);
   g_free (data);
   gtk_css_scanner_destroy (scanner);
 
@@ -2739,6 +2748,7 @@ gtk_css_provider_load_from_path_internal (GtkCssProvider  *css_provider,
   GScanner *scanner;
   const gchar *data;
   gsize length;
+  GFile *file;
   gboolean ret;
 
   priv = css_provider->priv;
@@ -2760,9 +2770,9 @@ gtk_css_provider_load_from_path_internal (GtkCssProvider  *css_provider,
   if (reset)
     gtk_css_provider_reset (css_provider);
 
-  scanner = gtk_css_scanner_new ();
-  scanner->input_name = path;
-  g_scanner_input_text (scanner, data, (guint) length);
+  file = g_file_new_for_path (path);
+  scanner = gtk_css_scanner_new (file, data, length);
+  g_object_unref (file);
 
   ret = parse_stylesheet (css_provider, scanner, error);
 



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