[Xfce4-commits] <midori:master> Speed-up JS generator function in Adblock

[Christian Dywan]

Updating branch refs/heads/master
         to f46f3f0be0ab648fde87dc4a737e8047a74b3a81 (commit)
       from 5f1cd907ad4ff7678c9204e44d938a4c8663c3fe (commit)

commit f46f3f0be0ab648fde87dc4a737e8047a74b3a81
Author: Paweł Forysiuk <tuxator at o2.pl>
Date:   Mon Jun 6 12:47:19 2011 -0400

    Speed-up JS generator function in Adblock

 extensions/adblock.c |   21 ++++++---------------
 1 files changed, 6 insertions(+), 15 deletions(-)

diff --git a/extensions/adblock.c b/extensions/adblock.c
index 47fb5b5..f271df2 100644
--- a/extensions/adblock.c
+++ b/extensions/adblock.c
@@ -720,10 +720,7 @@ static gchar*
 adblock_prepare_urihider_js (GList* uris)
 {
     GList* li = NULL;
-    gchar* out;
-    gchar* cmd = g_strdup ("");
-    gchar* tmp;
-    const char *js =
+    GString* js = g_string_new (
         "(function() {"
         "function getElementsByAttribute (strTagName, strAttributeName, arrAttributeValue) {"
         "    var arrElements = document.getElementsByTagName (strTagName);"
@@ -749,20 +746,14 @@ adblock_prepare_urihider_js (GList* uris)
         "        oElements[i].style.height = '0';"
         "    }"
         "};"
-        "var uris=new Array ();"
-        "%s"
-        "hideElementBySrc (uris);"
-        "})();";
+        "var uris=new Array ();");
 
     for (li = uris; li != NULL; li = g_list_next (li))
-    {
-        tmp = g_strdup_printf ("uris.push ('%s');%s", (gchar*)li->data, cmd);
-        katze_assign (cmd, tmp);
-    }
-    out = g_strdup_printf (js, cmd);
-    g_free (cmd);
+        g_string_append_printf (js, "uris.push ('%s');", (gchar*)li->data);
+
+    g_string_append (js, "hideElementBySrc (uris);})();");
 
-    return out;
+    return g_string_free (js, FALSE);
 }
 
 static void