Restore previous switch statement now that its logic is isolated

npa.js

@@ -154,24 +154,37 @@ function parseUrl (res, arg, urlparse) {
       https:     gitHost.https(),
       directUrl: gitHost.file("package.json")
     }
+    return res
   }
   // check the protocol, and then see if it's git or not
-  else if (/^git([+](https?|rsync|ftp|ssh|file))?:$/.test(urlparse.protocol) ||
-      (/^https?:$/.test(urlparse.protocol) && /[.]git$/.test(urlparse.pathname))) {
-    res.type = "git"
-    res.spec = arg.replace(/^git[+]/, "")
-  }
-  else if (/^https?:$/.test(urlparse.protocol)) {
-    res.type = "remote"
-    res.spec = arg
-  }
-  else if ("file:" === urlparse.protocol) {
-    res.type = "local"
-    res.spec = urlparse.pathname
-  }
-  else {
-    throw new Error("Unsupported URL Type: " + arg)
+  switch (urlparse.protocol) {
+    case "git:":
+    case "git+http:":
+    case "git+https:":
+    case "git+rsync:":
+    case "git+ftp:":
+    case "git+ssh:":
+    case "git+file:":
+      res.type = 'git'
+      res.spec = arg.replace(/^git\+/, '')
+      break
+
+    case 'http:':
+    case 'https:':
+      res.type = 'remote'
+      res.spec = arg
+      break
+
+    case 'file:':
+      res.type = 'local'
+      res.spec = urlparse.pathname
+      break;
+
+    default:
+      throw new Error('Unsupported URL Type: ' + arg)
+      break
   }
+
   return res
 }