Move tokenizer and parser functions into closures so they could be ev…

…aluated only once instead of for every run
css · Dec 8, 2012 · ffd3a25 · ffd3a25
1 parent 5034176
commit ffd3a25
Show file tree

Hide file tree

Showing 6 changed files with 52 additions and 28 deletions.
diff --git a/lib/gonzales.cssp.node.js b/lib/gonzales.cssp.node.js
@@ -1,4 +1,4 @@
-function srcToCSSP(s, rule, _needInfo) {
+var srcToCSSP = (function() {
 var TokenType = {
     StringSQ: 'StringSQ',
     StringDQ: 'StringDQ',
@@ -45,7 +45,8 @@ var TokenType = {
     DecimalNumber: 'DecimalNumber'
 };
 
-function getTokens(s) {
+var getTokens = (function() {
+
     var Punctuation,
         urlMode = false,
         blockMode = 0;
@@ -268,11 +269,14 @@ function getTokens(s) {
         }
     }
 
-    return _getTokens(s);
-}
+    return function(s) {
+        return _getTokens(s);
+    };
+
+}());
 // version: 1.0.0
 
-function getCSSPAST(_tokens, rule, _needInfo) {
+var getCSSPAST = (function() {
 
     var tokens,
         pos,
@@ -2281,9 +2285,13 @@ function getCSSPAST(_tokens, rule, _needInfo) {
         if (sc !== -1) tokens[sc].sc_last = i - 1;
     }
 
-    return _getAST(_tokens, rule, _needInfo);
-}
+    return function(_tokens, rule, _needInfo) {
+        return _getAST(_tokens, rule, _needInfo);
+    }
 
-    return getCSSPAST(getTokens(s), rule, _needInfo);
-}
+}());
+    return function(s, rule, _needInfo) {
+        return getCSSPAST(getTokens(s), rule, _needInfo);
+    }
+}());
 exports.srcToCSSP = srcToCSSP;
diff --git a/src/cssp.ast.shared.js b/src/cssp.ast.shared.js
@@ -1,6 +1,6 @@
 // version: 1.0.0
 
-function getCSSPAST(_tokens, rule, _needInfo) {
+var getCSSPAST = (function() {
 
     var tokens,
         pos,
@@ -2009,5 +2009,8 @@ function getCSSPAST(_tokens, rule, _needInfo) {
         if (sc !== -1) tokens[sc].sc_last = i - 1;
     }
 
-    return _getAST(_tokens, rule, _needInfo);
-}
+    return function(_tokens, rule, _needInfo) {
+        return _getAST(_tokens, rule, _needInfo);
+    }
+
+}());
diff --git a/src/gonzales.cssp.footer.js b/src/gonzales.cssp.footer.js
@@ -1,3 +1,4 @@
-
-    return getCSSPAST(getTokens(s), rule, _needInfo);
-}
+    return function(s, rule, _needInfo) {
+        return getCSSPAST(getTokens(s), rule, _needInfo);
+    }
+}());
diff --git a/src/gonzales.cssp.header.js b/src/gonzales.cssp.header.js
@@ -1 +1 @@
-function srcToCSSP(s, rule, _needInfo) {
+var srcToCSSP = (function() {
diff --git a/src/tokenizer.shared.js b/src/tokenizer.shared.js
@@ -44,7 +44,8 @@ var TokenType = {
     DecimalNumber: 'DecimalNumber'
 };
 
-function getTokens(s) {
+var getTokens = (function() {
+
     var Punctuation,
         urlMode = false,
         blockMode = 0;
@@ -267,5 +268,8 @@ function getTokens(s) {
         }
     }
 
-    return _getTokens(s);
-}
+    return function(s) {
+        return _getTokens(s);
+    };
+
+}());
diff --git a/web/gonzales.cssp.web.js b/web/gonzales.cssp.web.js
@@ -1,4 +1,4 @@
-function srcToCSSP(s, rule, _needInfo) {
+var srcToCSSP = (function() {
 var TokenType = {
     StringSQ: 'StringSQ',
     StringDQ: 'StringDQ',
@@ -45,7 +45,8 @@ var TokenType = {
     DecimalNumber: 'DecimalNumber'
 };
 
-function getTokens(s) {
+var getTokens = (function() {
+
     var Punctuation,
         urlMode = false,
         blockMode = 0;
@@ -268,11 +269,14 @@ function getTokens(s) {
         }
     }
 
-    return _getTokens(s);
-}
+    return function(s) {
+        return _getTokens(s);
+    };
+
+}());
 // version: 1.0.0
 
-function getCSSPAST(_tokens, rule, _needInfo) {
+var getCSSPAST = (function() {
 
     var tokens,
         pos,
@@ -2281,8 +2285,12 @@ function getCSSPAST(_tokens, rule, _needInfo) {
         if (sc !== -1) tokens[sc].sc_last = i - 1;
     }
 
-    return _getAST(_tokens, rule, _needInfo);
-}
+    return function(_tokens, rule, _needInfo) {
+        return _getAST(_tokens, rule, _needInfo);
+    }
 
-    return getCSSPAST(getTokens(s), rule, _needInfo);
-}
+}());
+    return function(s, rule, _needInfo) {
+        return getCSSPAST(getTokens(s), rule, _needInfo);
+    }
+}());