Chore: combine multiple RegEx replaces with one (fixes #6669) (#6661)

eslint · Jul 15, 2016 · 421e4bf · 421e4bf
1 parent 089ee2c
commit 421e4bf
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/lib/eslint.js b/lib/eslint.js
@@ -43,11 +43,8 @@ var assert = require("assert"),
 function parseBooleanConfig(string, comment) {
     var items = {};
 
-    // Collapse whitespace around : to make parsing easier
-    string = string.replace(/\s*:\s*/g, ":");
-
-    // Collapse whitespace around ,
-    string = string.replace(/\s*,\s*/g, ",");
+    // Collapse whitespace around `:` and `,` to make parsing easier
+    string = string.replace(/\s*([:,])\s*/g, "$1");
 
     string.split(/\s|,+/).forEach(function(name) {
         if (!name) {