Skip to content

Grab file name from file_info for accuracy

Rodrigo Muino Tomonari requested to merge github/fork/jsheely/master into master

The handle->filew out of ReadDirectoryChangesW grabs the closest match to the file but does not contain the actual file name

fs.watch('*/.jsx'); ex. /path/to/file/filename.jsx will match /path/to/file/filename.js if you use handle->filew

Using file_info->FileName it will return an event for filename.jsx or one for filename.js but at least in this case you will be able to properly filter the value in node since you have the correct file name to match against.

There is probably more we can do in order to ensure that the event doesn't leave libuv if it doesn't match the watched file path but at least this way we can accurately do something about it.

Merge request reports

Loading