sax - npm Package Compare versions

Comparing version 1.2.1 to 1.2.2

lib/sax.js

@@ -270,4 +270,2 @@ ;(function (sax) { // wrapper for non-node envs
   // XML allows all manner of ridiculous numbers and digits.
-  var number = '0124356789'
-  var letter = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ'
 
@@ -285,4 +283,2 @@ // (Letter | "_" | ":")
   whitespace = charClass(whitespace)
-  number = charClass(number)
-  letter = charClass(letter)
 
@@ -297,6 +293,6 @@ // http://www.w3.org/TR/REC-xml/#NT-NameStartChar
 
-  var nameBody = /[:_A-Za-z\u00C0-\u00D6\u00D8-\u00F6\u00F8-\u02FF\u0370-\u037D\u037F-\u1FFF\u200C-\u200D\u2070-\u218F\u2C00-\u2FEF\u3001-\uD7FF\uF900-\uFDCF\uFDF0-\uFFFD\u00B7\u0300-\u036F\u203F-\u2040\.\d-]/
+  var nameBody = /[:_A-Za-z\u00C0-\u00D6\u00D8-\u00F6\u00F8-\u02FF\u0370-\u037D\u037F-\u1FFF\u200C-\u200D\u2070-\u218F\u2C00-\u2FEF\u3001-\uD7FF\uF900-\uFDCF\uFDF0-\uFFFD\u00B7\u0300-\u036F\u203F-\u2040.\d-]/
 
   var entityStart = /[#:_A-Za-z\u00C0-\u00D6\u00D8-\u00F6\u00F8-\u02FF\u0370-\u037D\u037F-\u1FFF\u200C-\u200D\u2070-\u218F\u2C00-\u2FEF\u3001-\uD7FF\uF900-\uFDCF\uFDF0-\uFFFD]/
-  var entityBody = /[#:_A-Za-z\u00C0-\u00D6\u00D8-\u00F6\u00F8-\u02FF\u0370-\u037D\u037F-\u1FFF\u200C-\u200D\u2070-\u218F\u2C00-\u2FEF\u3001-\uD7FF\uF900-\uFDCF\uFDF0-\uFFFD\u00B7\u0300-\u036F\u203F-\u2040\.\d-]/
+  var entityBody = /[#:_A-Za-z\u00C0-\u00D6\u00D8-\u00F6\u00F8-\u02FF\u0370-\u037D\u037F-\u1FFF\u200C-\u200D\u2070-\u218F\u2C00-\u2FEF\u3001-\uD7FF\uF900-\uFDCF\uFDF0-\uFFFD\u00B7\u0300-\u036F\u203F-\u2040.\d-]/
 
@@ -313,10 +309,14 @@ quote = charClass(quote)
 
-  function isRegExp (c) {
-    return Object.prototype.toString.call(c) === '[object RegExp]'
+  function isMatch (regex, c) {
+    return regex.test(c)
   }
 
   function is (charclass, c) {
-    return isRegExp(charclass) ? !!c.match(charclass) : charclass[c]
+    return charclass[c]
   }
 
+  function notMatch (regex, c) {
+    return !isMatch(regex, c)
+  }
+
   function not (charclass, c) {
@@ -1005,5 +1005,7 @@ return !is(charclass, c)
       parser.c = c
+
       if (!c) {
         break
       }
+
       if (parser.trackPosition) {
@@ -1018,2 +1020,3 @@ parser.position++
       }
+
       switch (parser.state) {
@@ -1089,3 +1092,3 @@ case S.BEGIN:
             // wait for it...
-          } else if (is(nameStart, c)) {
+          } else if (isMatch(nameStart, c)) {
             parser.state = S.OPEN_TAG
@@ -1293,3 +1296,3 @@ parser.tagName = c
         case S.OPEN_TAG:
-          if (is(nameBody, c)) {
+          if (isMatch(nameBody, c)) {
             parser.tagName += c
@@ -1329,3 +1332,3 @@ } else {
             parser.state = S.OPEN_TAG_SLASH
-          } else if (is(nameStart, c)) {
+          } else if (isMatch(nameStart, c)) {
             parser.attribName = c
@@ -1349,3 +1352,3 @@ parser.attribValue = ''
             parser.state = S.ATTRIB_NAME_SAW_WHITE
-          } else if (is(nameBody, c)) {
+          } else if (isMatch(nameBody, c)) {
             parser.attribName += c
@@ -1373,3 +1376,3 @@ } else {
               openTag(parser)
-            } else if (is(nameStart, c)) {
+            } else if (isMatch(nameStart, c)) {
               parser.attribName = c
@@ -1418,3 +1421,3 @@ parser.state = S.ATTRIB_NAME
             parser.state = S.OPEN_TAG_SLASH
-          } else if (is(nameStart, c)) {
+          } else if (isMatch(nameStart, c)) {
             strictFail(parser, 'No whitespace between attributes')
@@ -1450,3 +1453,3 @@ parser.attribName = c
               continue
-            } else if (not(nameStart, c)) {
+            } else if (notMatch(nameStart, c)) {
               if (parser.script) {
@@ -1463,3 +1466,3 @@ parser.script += '</' + c
             closeTag(parser)
-          } else if (is(nameBody, c)) {
+          } else if (isMatch(nameBody, c)) {
             parser.tagName += c
@@ -1515,3 +1518,3 @@ } else if (parser.script) {
             parser.state = returnState
-          } else if (is(parser.entity.length ? entityBody : entityStart, c)) {
+          } else if (isMatch(parser.entity.length ? entityBody : entityStart, c)) {
             parser.entity += c
@@ -1539,2 +1542,3 @@ } else {
   /*! http://mths.be/fromcodepoint v0.1.0 by @mathias */
+  /* istanbul ignore next */
   if (!String.fromCodePoint) {
@@ -1581,2 +1585,3 @@ (function () {
       }
+      /* istanbul ignore next */
       if (Object.defineProperty) {
@@ -1583,0 +1588,0 @@ Object.defineProperty(String, 'fromCodePoint', {

package.json

@@ -5,7 +5,7 @@ {
   "author": "Isaac Z. Schlueter <[e-mail]> (http://blog.izs.me/)",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "main": "lib/sax.js",
   "license": "ISC",
   "scripts": {
-    "test": "tap test/*.js --cov",
+    "test": "tap test/*.js --cov -j4",
     "posttest": "standard -F test/*.js lib/*.js"
@@ -17,9 +17,8 @@ },
     "LICENSE",
-    "LICENSE-W3C.html",
     "README.md"
   ],
   "devDependencies": {
-    "standard": "^5.3.1",
-    "tap": "^5.2.0"
+    "standard": "^8.6.0",
+    "tap": "^10.0.2"
   }
 }

Fixed alerts

Unidentified License

License

(Experimental) Something that seems like a license was found, but its contents could not be matched with a known license

Found 1 instance in 1 package

Improved metrics

Number of low license alerts

1

decreased by-50%

Lines of code

1448

increased by0.07%

Worsened metrics

Total package byte prevSize

54814

decreased by-17.89%

Number of package files

4

decreased by-20%

No dependency changes