Commit 5c84953
path: remove unnecessary if statement
There is an `if`-statement in `normalizeString` (a helper function for
`path.normalize`) whose `else`-branch is never taken. This patch
removes it.
PR-URL: #22273
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent db79276 commit 5c84953
1 file changed
Lines changed: 9 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -77,18 +77,16 @@ function normalizeString(path, allowAboveRoot, separator, isPathSeparator) { | |||
| 77 | 77 | res.charCodeAt(res.length - 2) !== CHAR_DOT) { | |
| 78 | 78 | if (res.length > 2) { | |
| 79 | 79 | const lastSlashIndex = res.lastIndexOf(separator); | |
| 80 | - if (lastSlashIndex !== res.length - 1) { | ||
| 81 | - if (lastSlashIndex === -1) { | ||
| 82 | - res = ''; | ||
| 83 | - lastSegmentLength = 0; | ||
| 84 | - } else { | ||
| 85 | - res = res.slice(0, lastSlashIndex); | ||
| 86 | - lastSegmentLength = res.length - 1 - res.lastIndexOf(separator); | ||
| 87 | - } | ||
| 88 | - lastSlash = i; | ||
| 89 | - dots = 0; | ||
| 90 | - continue; | ||
| 80 | + if (lastSlashIndex === -1) { | ||
| 81 | + res = ''; | ||
| 82 | + lastSegmentLength = 0; | ||
| 83 | + } else { | ||
| 84 | + res = res.slice(0, lastSlashIndex); | ||
| 85 | + lastSegmentLength = res.length - 1 - res.lastIndexOf(separator); | ||
| 91 | 86 | } | |
| 87 | + lastSlash = i; | ||
| 88 | + dots = 0; | ||
| 89 | + continue; | ||
| 92 | 90 | } else if (res.length === 2 || res.length === 1) { | |
| 93 | 91 | res = ''; | |
| 94 | 92 | lastSegmentLength = 0; | |
0 commit comments