Commit 0bd9d9e
os: avoid unnecessary usage of var
The `var` keyword is known to be problematic and is not needed here,
so better to use the `let` keyword for variable declarations.
PR-URL: #42563
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 93ffc55 commit 0bd9d9e
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -171,7 +171,7 @@ platform[SymbolToPrimitive] = () => process.platform; | |||
| 171 | 171 | * @returns {string} | |
| 172 | 172 | */ | |
| 173 | 173 | function tmpdir() { | |
| 174 | - var path; | ||
| 174 | + let path; | ||
| 175 | 175 | if (isWindows) { | |
| 176 | 176 | path = process.env.TEMP || | |
| 177 | 177 | process.env.TMP || | |
@@ -223,7 +223,7 @@ function getCIDR(address, netmask, family) { | |||
| 223 | 223 | } | |
| 224 | 224 | ||
| 225 | 225 | const parts = StringPrototypeSplit(netmask, split); | |
| 226 | - for (var i = 0; i < parts.length; i++) { | ||
| 226 | + for (let i = 0; i < parts.length; i++) { | ||
| 227 | 227 | if (parts[i] !== '') { | |
| 228 | 228 | const binary = NumberParseInt(parts[i], range); | |
| 229 | 229 | const tmp = countBinaryOnes(binary); | |
@@ -261,7 +261,7 @@ function networkInterfaces() { | |||
| 261 | 261 | ||
| 262 | 262 | if (data === undefined) | |
| 263 | 263 | return result; | |
| 264 | - for (var i = 0; i < data.length; i += 7) { | ||
| 264 | + for (let i = 0; i < data.length; i += 7) { | ||
| 265 | 265 | const name = data[i]; | |
| 266 | 266 | const entry = { | |
| 267 | 267 | address: data[i + 1], | |
0 commit comments