Minimized JavaScript: keyword "var" only once per scope
authorKai Moritz <kai@coolibri.de>
Mon, 24 Sep 2012 11:53:40 +0000 (13:53 +0200)
committerKai Moritz <kai@coolibri.de>
Tue, 26 Feb 2013 13:40:02 +0000 (14:40 +0100)
openx.js

index 94a3bb4..eb85805 100644 (file)
--- a/openx.js
+++ b/openx.js
@@ -4,30 +4,35 @@
 
 (function( openx, $, undefined ) {
 
-  var id;
-  var node;
+  var
 
-  var count = 0;
-  var slots = {};
-  var ads = [];
+  id,
+  node,
+
+  count = 0,
+  slots = {},
+  ads = [];
 
 
   openx.show_ads = function(server, zones) {
 
+    var
+    domain = document.location.protocol == 'https:' ? 'https://' + server + ':8443':'http://' + server,
+    src = domain;
+
     document.write = render;
     document.writeln = render;
 
-    var domain = document.location.protocol == 'https:' ? 'https://' + server + ':8443':'http://' + server;
-
-    var src = domain;
     src += "/www/delivery/spc.php?zones=";
 
     /** Only fetch banners, that are really included in this page */
     $('.oa').each(function() {
-      var node = $(this);
-      for(var name in zones) {
+      var
+      node = $(this),
+      name, id;
+      for(name in zones) {
         if (node.hasClass(name)) {
-          var id = 'oa_' + ++count;
+          id = 'oa_' + ++count;
           slots[id] = node;
           src += escape(id + '=' + zones[name] + "|");
         }
@@ -72,9 +77,7 @@
 
     // node.append(id + ": " + node.attr('class'));
 
-    var result;
-    var src;
-    var inline;
+    var result, src, inline;
 
     while ((result = /<script/i.exec(OA_output[id])) != null) {
       node.append(OA_output[id].slice(0,result.index));
     if (id == undefined)
       return;
 
-    var str = "";
-    for (var i=0; i < arguments.length; i++)
+    var
+    str = "",
+    i;
+
+    for (i=0; i < arguments.length; i++)
       str += arguments[i];
 
     OA_output[id] = str + OA_output[id];