Small performance improvement on escapeId

Author: dougwilson

HISTORY.md

@@ -2,6 +2,7 @@ unreleased
 ==========
 
   * Add `.toSqlString()` escape overriding
+  * Small performance improvement on `escapeId`
 
 2.2.0 / 2016-11-01
 ==================

lib/SqlString.js

@@ -1,5 +1,7 @@
 var SqlString  = exports;
 
+var ID_GLOBAL_REGEXP    = /`/g;
+var QUAL_GLOBAL_REGEXP  = /\./g;
 var CHARS_GLOBAL_REGEXP = /[\0\b\t\n\r\x1a\"\'\\]/g; // eslint-disable-line no-control-regex
 var CHARS_ESCAPE_MAP    = {
   '\0'   : '\\0',
@@ -22,13 +24,11 @@ SqlString.escapeId = function escapeId(val, forbidQualified) {
     }
 
     return sql;
+  } else if (forbidQualified) {
+    return '`' + String(val).replace(ID_GLOBAL_REGEXP, '``') + '`';
+  } else {
+    return '`' + String(val).replace(ID_GLOBAL_REGEXP, '``').replace(QUAL_GLOBAL_REGEXP, '`.`') + '`';
   }
-
-  if (forbidQualified) {
-    return '`' + String(val).replace(/`/g, '``') + '`';
-  }
-
-  return '`' + String(val).replace(/`/g, '``').replace(/\./g, '`.`') + '`';
 };
 
 SqlString.escape = function escape(val, stringifyObjects, timeZone) {