fs: refactor to use ES2020 syntax

xtx1130 · danielleadams · commit 2d165652f16a · 2022-04-23T22:25:58.000-04:00
PR-URL: #41903 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/lib/fs.js b/lib/fs.js
@@ -2516,7 +2516,7 @@ function realpathSync(p, options) {
       }
       resolvedLink = pathModule.resolve(previous, linkTarget);
 
-      if (cache) cache.set(base, resolvedLink);
+      cache?.set(base, resolvedLink);
       if (!isWindows) seenLinks[id] = linkTarget;
     }
 
@@ -3041,8 +3041,7 @@ ObjectDefineProperties(fs, {
     configurable: true,
     enumerable: true,
     get() {
-      if (promises === null)
-        promises = require('internal/fs/promises').exports;
+      promises ??= require('internal/fs/promises').exports;
       return promises;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -2516,7 +2516,7 @@ function realpathSync(p, options) {`
`2516`	`2516`	`}`
`2517`	`2517`	`resolvedLink = pathModule.resolve(previous, linkTarget);`
`2518`	`2518`
`2519`		`- if (cache) cache.set(base, resolvedLink);`
	`2519`	`+ cache?.set(base, resolvedLink);`
`2520`	`2520`	`if (!isWindows) seenLinks[id] = linkTarget;`
`2521`	`2521`	`}`
`2522`	`2522`
`@@ -3041,8 +3041,7 @@ ObjectDefineProperties(fs, {`
`3041`	`3041`	`configurable: true,`
`3042`	`3042`	`enumerable: true,`
`3043`	`3043`	`get() {`
`3044`		`- if (promises === null)`
`3045`		`- promises = require('internal/fs/promises').exports;`
	`3044`	`+ promises ??= require('internal/fs/promises').exports;`
`3046`	`3045`	`return promises;`
`3047`	`3046`	`}`
`3048`	`3047`	`}`