watch: clarify that the fileName parameter can be null

Add a comment to clarify that the `fileName` parameter can be `null` if the file name cannot be determined.
nodejs · Dec 28, 2023 · fc3bdab · fc3bdab
1 parent 9db4bf4
commit fc3bdab
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/internal/watch_mode/files_watcher.js b/lib/internal/watch_mode/files_watcher.js
@@ -97,8 +97,11 @@ class FilesWatcher extends EventEmitter {
       return;
     }
     const watcher = watch(path, { recursive, signal: this.#signal });
-    watcher.on('change', (eventType, fileName) => this
-      .#onChange(recursive ? resolve(path, fileName ?? '') : path));
+    watcher.on('change', (eventType, fileName) => {
+      // `fileName` can be `null` on Windows. See
+      // https://github.com/nodejs/node/pull/49891#issuecomment-1744673430.
+      this.#onChange(recursive ? resolve(path, fileName ?? '') : path);
+    });
     this.#watchers.set(path, { handle: watcher, recursive });
     if (recursive) {
       this.#removeWatchedChildren(path);