Don't use capturing groups during default preprocessing

We aren't using backreferences during default preprocessing,
so we don't need to capture any groups.

Signed-off-by: David A. Wheeler <[email protected]>

Author: david-a-wheeler

docs/labs/checker.js

@@ -41,9 +41,12 @@ let preprocessRegexes = [
   [/[ \t]+\\s\+[ \t]+/g, '\\s+'],
 
   // 1+ spaces/tabs are instead interpreted as \s* (0+ whitespace)
-  // The (\\s\*)? expressions before and after it are an optimization -
+  // The (?:\\s\*)? expressions before and after it are an optimization -
   // if you use \s* next to spaces/tabs, they coalesce for speed.
-  [/(\\s\*)?[ \t]+(\\s\*)?/g, '\\s*']
+  // We use non-capturing groups (?:...) for speed. Preprocessing is only
+  // done once on startup, so speed isn't important, but it's often good
+  // to avoid capturing groups when you don't need to.
+  [/(?:\\s\*)?[ \t]+(?:\\s\*)?/g, '\\s*']
 ];
 
 /**