Using both lstat and stat in onWatchEvent was unnecessary

webpack · Mar 9, 2024 · 5b22101 · 5b22101
1 parent 289ce5a
commit 5b22101
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/lib/DirectoryWatcher.js b/lib/DirectoryWatcher.js
@@ -393,7 +393,7 @@ class DirectoryWatcher extends EventEmitter {
 				if (this.closed) return;
 				this._activeEvents.set(filename, false);
 
-				const handleStats = (err, stats) => {
+				fs.lstat(filePath, (err, stats) => {
 					if (this.closed) return;
 					if (this._activeEvents.get(filename) === true) {
 						process.nextTick(checkStats);
@@ -439,15 +439,8 @@ class DirectoryWatcher extends EventEmitter {
 							false,
 							eventType
 						);
-						if (
-							this.watcherManager.options.followSymlinks &&
-							stats.isSymbolicLink()
-						) {
-							fs.stat(filePath, handleStats);
-						}
 					}
-				};
-				fs.lstat(filePath, handleStats);
+				});
 			};
 			process.nextTick(checkStats);
 		} else {