url: group slashed protocols by protocol name
authornettofarah <nettofarah@gmail.com>
Tue, 23 Feb 2016 07:47:22 +0000 (23:47 -0800)
committerMyles Borins <mborins@us.ibm.com>
Mon, 21 Mar 2016 19:57:59 +0000 (12:57 -0700)
Reorder slashed protocols so they are grouped by protocol name. This is
done so it doesn't look like we're duplicating protocol names at the
bottom of the list.

PR-URL: https://github.com/nodejs/node/pull/5380
Reviewed-By: Evan Lucas <evanlucas@me.com>
lib/url.js

index d9be77a..016acd0 100644 (file)
@@ -65,14 +65,14 @@ const hostlessProtocol = {
 // protocols that always contain a // bit.
 const slashedProtocol = {
   'http': true,
-  'https': true,
-  'ftp': true,
-  'gopher': true,
-  'file': true,
   'http:': true,
+  'https': true,
   'https:': true,
+  'ftp': true,
   'ftp:': true,
+  'gopher': true,
   'gopher:': true,
+  'file': true,
   'file:': true
 };
 const querystring = require('querystring');