Perf: tweaked typorgapher checks

lib/rules_text/replace.js

@@ -3,6 +3,9 @@
 'use strict';
 
 
+var COPY_RE = /\((c|tm|r|p)\)/i;
+var RARE_RE = /\+-|\.\.|\?\?\?\?|!!!!|,,|--/;
+
 module.exports = function replace(t, state) {
   var i, token, text,
       tokens = state.tokens,
@@ -13,7 +16,7 @@ module.exports = function replace(t, state) {
     if (token.type === 'text') {
       text = token.content;
 
-      if (text.indexOf('(') >= 0) {
+      if (COPY_RE.test(text)) {
         if (options.copyright) {
           text = text.replace(/\(c\)/gi, '©');
         }
@@ -28,21 +31,19 @@ module.exports = function replace(t, state) {
         }
       }
 
-      if (options.plusminus && text.indexOf('+-') >= 0) {
+      if (RARE_RE.test(text)) {
+        if (options.plusminus) {
           text = text.replace(/\+-/g, '±');
         }
-      if (options.ellipsis && text.indexOf('..') >= 0) {
+        if (options.ellipsis) {
           // .., ..., ....... -> …
           // but ?..... & !..... -> ?.. & !..
           text = text.replace(/\.{2,}/g, '…').replace(/([?!])…/g, '$1..');
         }
-      if (options.dupes &&
+        if (options.dupes) {
-          (text.indexOf('????') >= 0 ||
-           text.indexOf('!!!!') >= 0 ||
-           text.indexOf(',,') >= 0)) {
           text = text.replace(/([?!]){4,}/g, '$1$1$1').replace(/,{2,}/g, ',');
         }
-      if (options.dashes && text.indexOf('--') >= 0) {
+        if (options.dashes) {
           text = text
                   // em-dash
                   .replace(/(^|[^-])---([^-]|$)/mg, '$1\u2014$2')
@@ -50,6 +51,7 @@ module.exports = function replace(t, state) {
                   .replace(/(^|\s)--(\s|$)/mg, '$1\u2013$2')
                   .replace(/(^|[^-\s])--([^-\s]|$)/mg, '$1\u2013$2');
         }
+      }
 
       token.content = text;
     }

lib/rules_text/smartquotes.js

@@ -3,20 +3,21 @@
 'use strict';
 
 
-var quoteReg = /['"]/g;
+var QUOTE_TEST_RE = /['"]/;
-var punctReg = /[-\s()\[\]]/;
+var QUOTE_RE = /['"]/g;
-var apostrophe = '’';
+var PUNCT_RE = /[-\s()\[\]]/;
+var APOSTROPHE = '’';
 
 // This function returns true if the character at `pos`
 // could be inside a word.
 function isLetter(str, pos) {
   if (pos < 0 || pos >= str.length) { return false; }
-  return !punctReg.test(str[pos]);
+  return !PUNCT_RE.test(str[pos]);
 }
 
 
 function addQuote(obj, tokenId, posId, str) {
-  if (!obj[tokenId]) { obj[tokenId] = {}; }
+  if (typeof obj[tokenId] === 'undefined') { obj[tokenId] = {}; }
   obj[tokenId][posId] = str;
 }
 
@@ -31,6 +32,9 @@ module.exports = function smartquotes(typographer, state) {
 
   for (i = 0; i < tokens.length; i++) {
     token = tokens[i];
+
+    if (token.type !== 'text' || QUOTE_TEST_RE.test(token.text)) { continue; }
+
     thisLevel = tokens[i].level;
 
     for (j = stack.length - 1; j >= 0; j--) {
@@ -38,14 +42,13 @@ module.exports = function smartquotes(typographer, state) {
     }
     stack.length = j + 1;
 
-    if (token.type === 'text') {
     text = token.content;
     pos = 0;
     max = text.length;
 
     while (pos < max) {
-        quoteReg.lastIndex = pos;
+      QUOTE_RE.lastIndex = pos;
-        t = quoteReg.exec(text);
+      t = QUOTE_RE.exec(text);
       if (!t) { break; }
 
       lastSpace = !isLetter(text, t.index - 1);
@@ -56,7 +59,7 @@ module.exports = function smartquotes(typographer, state) {
       if (!nextSpace && !lastSpace) {
         // middle word
         if (isSingle) {
-            addQuote(replace, i, t.index, apostrophe);
+          addQuote(replace, i, t.index, APOSTROPHE);
         }
         continue;
       }
@@ -92,20 +95,19 @@ module.exports = function smartquotes(typographer, state) {
           level: thisLevel
         });
       } else if (canClose && isSingle) {
-          addQuote(replace, i, t.index, apostrophe);
+        addQuote(replace, i, t.index, APOSTROPHE);
-        }
       }
     }
   }
 
   fn = function(str, pos) {
-    if (!replace[i][pos]) { return str; }
+    if (typeof replace[i][pos] === 'undefined') { return str; }
     return replace[i][pos];
   };
 
   for (i = 0; i < tokens.length; i++) {
-    if (!replace[i]) { continue; }
+    if (typeof replace[i] === 'undefined') { continue; }
-    quoteReg.lastIndex = 0;
+    QUOTE_RE.lastIndex = 0;
-    tokens[i].content = tokens[i].content.replace(quoteReg, fn);
+    tokens[i].content = tokens[i].content.replace(QUOTE_RE, fn);
   }
 };