chore: remove confusing comment

vitejs · Nov 8, 2022 · 343e206 · 343e206
1 parent c1380cd
commit 343e206
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 9 deletions.
diff --git a/packages/vite/src/node/plugins/resolve.ts b/packages/vite/src/node/plugins/resolve.ts
@@ -531,9 +531,6 @@ function tryResolveFile(
   tryPrefix?: string,
   skipPackageJson?: boolean
 ): string | undefined {
-  // #2051 if we don't have read permission on a directory, existsSync() still
-  // works and will result in massively slow subsequent checks (which are
-  // unnecessary in the first place)
   if (isFileReadable(file)) {
     if (!fs.statSync(file).isDirectory()) {
       return getRealPath(file, options.preserveSymlinks) + postfix

diff --git a/packages/vite/src/node/utils.ts b/packages/vite/src/node/utils.ts
@@ -535,12 +535,6 @@ export function writeFile(
   fs.writeFileSync(filename, content)
 }
 
-/**
- * Use fs.statSync(filename) instead of fs.existsSync(filename)
- * #2051 if we don't have read permission on a directory, existsSync() still
- * works and will result in massively slow subsequent checks (which are
- * unnecessary in the first place)
- */
 export function isFileReadable(filename: string): boolean {
   try {
     const stat = fs.statSync(filename, { throwIfNoEntry: false })