From 6d09b1e4f48dab56f07443a8d4baabfa33be47ae Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 3 Nov 2016 11:32:06 -0700 Subject: [PATCH] benchmark,lib,test,tools: remove unneeded . escape The `.` character does not need to be escaped when it appears inside a regular expression character class. This removes instances of unnecessary escapes of the `.` character. This also removes a few unnecessary escapes of the `(` and `)` characters within character classes too. --- benchmark/_http-benchmarkers.js | 2 +- lib/_tls_wrap.js | 2 +- test/parallel/test-repl.js | 2 +- tools/doc/json.js | 8 ++++---- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/benchmark/_http-benchmarkers.js b/benchmark/_http-benchmarkers.js index ca8b4625066f67..581bed6b2dc9e6 100644 --- a/benchmark/_http-benchmarkers.js +++ b/benchmark/_http-benchmarkers.js @@ -37,7 +37,7 @@ AutocannonBenchmarker.prototype.processResults = function(output) { function WrkBenchmarker() { this.name = 'wrk'; - this.regexp = /Requests\/sec:[ \t]+([0-9\.]+)/; + this.regexp = /Requests\/sec:[ \t]+([0-9.]+)/; const result = child_process.spawnSync('wrk', ['-h']); this.present = !(result.error && result.error.code === 'ENOENT'); } diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js index c31ae76651c0ab..478fd20188196d 100644 --- a/lib/_tls_wrap.js +++ b/lib/_tls_wrap.js @@ -919,7 +919,7 @@ Server.prototype.addContext = function(servername, context) { } var re = new RegExp('^' + - servername.replace(/([\.^$+?\-\\[\]{}])/g, '\\$1') + servername.replace(/([.^$+?\-\\[\]{}])/g, '\\$1') .replace(/\*/g, '[^.]*') + '$'); this._contexts.push([re, tls.createSecureContext(context).context]); diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js index a6603332aad672..29c85e36076dbd 100644 --- a/test/parallel/test-repl.js +++ b/test/parallel/test-repl.js @@ -72,7 +72,7 @@ function error_test() { if (read_buffer !== client_unix.expect) { var expect = client_unix.expect; if (expect === prompt_multiline) - expect = /[\.]{3} /; + expect = /[.]{3} /; assert.ok(read_buffer.match(expect)); console.error('match'); } diff --git a/tools/doc/json.js b/tools/doc/json.js index a194c7f7231423..a782c54028d756 100644 --- a/tools/doc/json.js +++ b/tools/doc/json.js @@ -545,12 +545,12 @@ function deepCopy_(src) { // these parse out the contents of an H# tag var eventExpr = /^Event(?::|\s)+['"]?([^"']+).*$/i; var classExpr = /^Class:\s*([^ ]+).*?$/i; -var propExpr = /^(?:property:?\s*)?[^\.]+\.([^ \.\(\)]+)\s*?$/i; -var braceExpr = /^(?:property:?\s*)?[^\.\[]+(\[[^\]]+\])\s*?$/i; +var propExpr = /^(?:property:?\s*)?[^.]+\.([^ .()]+)\s*?$/i; +var braceExpr = /^(?:property:?\s*)?[^.\[]+(\[[^\]]+\])\s*?$/i; var classMethExpr = - /^class\s*method\s*:?[^\.]+\.([^ \.\(\)]+)\([^\)]*\)\s*?$/i; + /^class\s*method\s*:?[^.]+\.([^ .()]+)\([^)]*\)\s*?$/i; var methExpr = - /^(?:method:?\s*)?(?:[^\.]+\.)?([^ \.\(\)]+)\([^\)]*\)\s*?$/i; + /^(?:method:?\s*)?(?:[^.]+\.)?([^ .()]+)\([^)]*\)\s*?$/i; var newExpr = /^new ([A-Z][a-zA-Z]+)\([^\)]*\)\s*?$/; var paramExpr = /\((.*)\);?$/;