sax 0.3.4 → 0.3.5

package/lib/sax.js

@@ -234,14 +234,37 @@ var whitespace = "\r\n\t "
   , nameBody = nameStart+number+"-."
   , quote = "'\""
   , entity = number+letter+"#"
+  , attribEnd = whitespace + ">"
   , CDATA = "[CDATA["
   , DOCTYPE = "DOCTYPE"
   , XML_NAMESPACE = "http://www.w3.org/XML/1998/namespace"
   , XMLNS_NAMESPACE = "http://www.w3.org/2000/xmlns/"
   , rootNS = { xml: XML_NAMESPACE, xmlns: XMLNS_NAMESPACE }
 
-function is (charclass, c) { return charclass.indexOf(c) !== -1 }
-function not (charclass, c) { return !is(charclass, c) }
+// turn all the string character sets into character class objects.
+whitespace = charClass(whitespace)
+number = charClass(number)
+letter = charClass(letter)
+nameStart = charClass(nameStart)
+nameBody = charClass(nameBody)
+quote = charClass(quote)
+entity = charClass(entity)
+attribEnd = charClass(attribEnd)
+
+function charClass (str) {
+  return str.split("").reduce(function (s, c) {
+    s[c] = true
+    return s
+  }, {})
+}
+
+function is (charclass, c) {
+  return charclass[c]
+}
+
+function not (charclass, c) {
+  return !charclass[c]
+}
 
 var S = 0
 sax.STATE =
@@ -905,7 +928,7 @@ function write (chunk) {
       continue
 
       case S.ATTRIB_VALUE_UNQUOTED:
-        if (not(whitespace+">",c)) {
+        if (not(attribEnd,c)) {
           if (c === "&") parser.state = S.ATTRIB_VALUE_ENTITY_U
           else parser.attribValue += c
           continue

package/package.json

@@ -1,7 +1,7 @@
 { "name" : "sax"
 , "description": "An evented streaming XML parser in JavaScript"
 , "author" : "Isaac Z. Schlueter <i@izs.me> (http://blog.izs.me/)"
-, "version" : "0.3.4"
+, "version" : "0.3.5"
 , "main" : "lib/sax.js"
 , "license" : { "type": "MIT"
               , "url": "https://raw.github.com/isaacs/sax-js/master/LICENSE" }

package/test/index.js

@@ -1,5 +1,5 @@
-
-var util = require("util")
+var globalsBefore = JSON.stringify(Object.keys(global))
+  , util = require("util")
   , assert = require("assert")
   , fs = require("fs")
   , path = require("path")
@@ -47,7 +47,7 @@ exports.test = function test (options) {
 
 if (module === require.main) {
   var running = true
-  , failures = 0
+    , failures = 0
 
   function fail (file, er) {
     util.error("Failed: "+file)
@@ -66,6 +66,14 @@ if (module === require.main) {
       // run this test.
       try {
         require(path.resolve(__dirname, file))
+        var globalsAfter = JSON.stringify(Object.keys(global))
+        if (globalsAfter !== globalsBefore) {
+          var er = new Error("new globals introduced\n"+
+                             "expected: "+globalsBefore+"\n"+
+                             "actual:   "+globalsAfter)
+          globalsBefore = globalsAfter
+          throw er
+        }
         console.log("ok " + (++i) + " - " + file)
       } catch (er) {
         console.log("not ok "+ (++i) + " - " + file)