From ce548073ab4e6e395040d2a52dbe4661f02b1e4c Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sat, 5 Nov 2016 19:08:31 -0700
Subject: [PATCH] lib,test: remove unneeded escaping of /

The `/` character does not need to be escaped when occurring inside a
character class in a regular expression. Remove such instances of
escaping in the code base.
---
 lib/fs.js                                           | 8 ++++----
 lib/url.js                                          | 2 +-
 test/debugger/test-debugger-repl-break-in-module.js | 6 +++---
 test/parallel/test-require-json.js                  | 2 +-
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/lib/fs.js b/lib/fs.js
index b8524c58841bb8..e22838da156fb2 100644
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -1436,13 +1436,13 @@ fs.unwatchFile = function(filename, listener) {
 // Regexp that finds the next portion of a (partial) path
 // result is [base_with_slash, base], e.g. ['somedir/', 'somedir']
 const nextPartRe = isWindows ?
-  /(.*?)(?:[\/\\]+|$)/g :
-  /(.*?)(?:[\/]+|$)/g;
+  /(.*?)(?:[/\\]+|$)/g :
+  /(.*?)(?:[/]+|$)/g;
 
 // Regex to find the device root, including trailing slash. E.g. 'c:\\'.
 const splitRootRe = isWindows ?
-  /^(?:[a-zA-Z]:|[\\\/]{2}[^\\\/]+[\\\/][^\\\/]+)?[\\\/]*/ :
-  /^[\/]*/;
+  /^(?:[a-zA-Z]:|[\\/]{2}[^\\/]+[\\/][^\\/]+)?[\\/]*/ :
+  /^[/]*/;
 
 function encodeRealpathResult(result, options, err) {
   if (!options || !options.encoding || options.encoding === 'utf8' || err)
diff --git a/lib/url.js b/lib/url.js
index 201ebfedcc5cda..a199f4b581dcd2 100644
--- a/lib/url.js
+++ b/lib/url.js
@@ -204,7 +204,7 @@ Url.prototype.parse = function(url, parseQueryString, slashesDenoteHost) {
   // user@server is *always* interpreted as a hostname, and url
   // resolution will treat //foo/bar as host=foo,path=bar because that's
   // how the browser resolves relative URLs.
-  if (slashesDenoteHost || proto || /^\/\/[^@\/]+@[^@\/]+/.test(rest)) {
+  if (slashesDenoteHost || proto || /^\/\/[^@/]+@[^@/]+/.test(rest)) {
     var slashes = rest.charCodeAt(0) === 47/*/*/ &&
                   rest.charCodeAt(1) === 47/*/*/;
     if (slashes && !(proto && hostlessProtocol[proto])) {
diff --git a/test/debugger/test-debugger-repl-break-in-module.js b/test/debugger/test-debugger-repl-break-in-module.js
index d855dc64f4aa25..91782ae6e3ef28 100644
--- a/test/debugger/test-debugger-repl-break-in-module.js
+++ b/test/debugger/test-debugger-repl-break-in-module.js
@@ -20,7 +20,7 @@ repl.addTest('sb(")^$*+?}{|][(.js\\\\", 1)', [
 
 // continue - the breakpoint should be triggered
 repl.addTest('c', [
-  /break in .*[\\\/]mod\.js:2/,
+  /break in .*[\\/]mod\.js:2/,
   /1/, /2/, /3/, /4/
 ]);
 
@@ -42,7 +42,7 @@ repl.addTest('restart', [].concat(
 
 // continue - the breakpoint should be triggered
 repl.addTest('c', [
-  /break in .*[\\\/]mod\.js:2/,
+  /break in .*[\\/]mod\.js:2/,
   /1/, /2/, /3/, /4/
 ]);
 
@@ -53,7 +53,7 @@ repl.addTest('cb("mod.js", 2)', [
 ]);
 
 repl.addTest('c', [
-  /break in .*[\\\/]main\.js:4/,
+  /break in .*[\\/]main\.js:4/,
   /2/, /3/, /4/, /5/, /6/
 ]);
 
diff --git a/test/parallel/test-require-json.js b/test/parallel/test-require-json.js
index be9f5f0aaad19e..75cdb98855e6ee 100644
--- a/test/parallel/test-require-json.js
+++ b/test/parallel/test-require-json.js
@@ -6,7 +6,7 @@ var assert = require('assert');
 try {
   require(path.join(common.fixturesDir, 'invalid.json'));
 } catch (err) {
-  var re = /test[\/\\]fixtures[\/\\]invalid.json: Unexpected string/;
+  var re = /test[/\\]fixtures[/\\]invalid.json: Unexpected string/;
   var i = err.message.match(re);
   assert.notStrictEqual(null, i, 'require() json error should include path');
 }