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

anonrig · H4ad · commit fccbb4114dc7 · 2023-09-17T20:58:01.000-03:00
Backport-PR-URL: nodejs#48658
diff --git a/lib/fs.js b/lib/fs.js
@@ -473,7 +473,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
`@@ -473,7 +473,8 @@ function readFileSync(path, options) {`
`473`	`473`
`474`	`474`	`// TODO(@anonrig): Do not handle file descriptor ownership for now.`
`475`	`475`	`if (!isUserFd && (options.encoding === 'utf8' \|\| options.encoding === 'utf-8')) {`
`476`		`- return readFileSyncUtf8(getValidatedPath(path), stringToFlags(options.flag));`
	`476`	`+ path = getValidatedPath(path);`
	`477`	`+ return readFileSyncUtf8(pathModule.toNamespacedPath(path), stringToFlags(options.flag));`
`477`	`478`	`}`
`478`	`479`
`479`	`480`	`const fd = isUserFd ? path : fs.openSync(path, options.flag, 0o666);`