fixup! fs: add a fast-path for readFileSync utf-8

anonrig · H4ad · commit cc7b5525b033 · 2023-12-01T23:22:35.000-03:00
Backport-PR-URL: nodejs#48658
diff --git a/lib/fs.js b/lib/fs.js
@@ -466,7 +466,8 @@ function readFileSync(path, options) {
 
   // TODO(@anonrig): Do not handle file descriptor ownership for now.
   if (!isUserFd && (options.encoding === 'utf8' || options.encoding === 'utf-8')) {
-    return readFileSyncUtf8(getValidatedPath(path), stringToFlags(options.flag));
+    path = getValidatedPath(path);
+    return readFileSyncUtf8(pathModule.toNamespacedPath(path), stringToFlags(options.flag));
   }
 
   const fd = isUserFd ? path : fs.openSync(path, options.flag, 0o666);

Original file line number	Diff line number	Diff line change
`@@ -466,7 +466,8 @@ function readFileSync(path, options) {`
`466`	`466`
`467`	`467`	`// TODO(@anonrig): Do not handle file descriptor ownership for now.`
`468`	`468`	`if (!isUserFd && (options.encoding === 'utf8' \|\| options.encoding === 'utf-8')) {`
`469`		`- return readFileSyncUtf8(getValidatedPath(path), stringToFlags(options.flag));`
	`469`	`+ path = getValidatedPath(path);`
	`470`	`+ return readFileSyncUtf8(pathModule.toNamespacedPath(path), stringToFlags(options.flag));`
`470`	`471`	`}`
`471`	`472`
`472`	`473`	`const fd = isUserFd ? path : fs.openSync(path, options.flag, 0o666);`