path: move branch to the correct location

BridgeAR · targos · commit 5b5c8196c3f8 · 2019-07-20T11:10:25.000+02:00
This code branch only makes sense when i === length. Otherwise it'll already be handled. PR-URL: #28556 Fixes: #28549 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Weijia Wang <starkwang@126.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
diff --git a/lib/path.js b/lib/path.js
@@ -518,11 +518,11 @@ const win32 = {
           lastCommonSep = 3;
         }
       }
+      if (lastCommonSep === -1)
+        lastCommonSep = 0;
     }
 
     let out = '';
-    if (lastCommonSep === -1)
-      lastCommonSep = 0;
     // Generate the relative path based on the path difference between `to` and
     // `from`
     for (i = fromStart + lastCommonSep + 1; i <= fromEnd; ++i) {

Original file line number	Diff line number	Diff line change
`@@ -518,11 +518,11 @@ const win32 = {`
`518`	`518`	`lastCommonSep = 3;`
`519`	`519`	`}`
`520`	`520`	`}`
	`521`	`+ if (lastCommonSep === -1)`
	`522`	`+ lastCommonSep = 0;`
`521`	`523`	`}`
`522`	`524`
`523`	`525`	`let out = '';`
`524`		`- if (lastCommonSep === -1)`
`525`		`- lastCommonSep = 0;`
`526`	`526`	// Generate the relative path based on the path difference between `to` and
`527`	`527`	// `from`
`528`	`528`	`for (i = fromStart + lastCommonSep + 1; i <= fromEnd; ++i) {`