Skip to content

Comments

Do not stat a path if not recursive.#256

Merged
JohnTitor merged 2 commits intonotify-rs:mainfrom
stuhood:stuhood/dont-stat-for-non-recursive
Jul 16, 2020
Merged

Do not stat a path if not recursive.#256
JohnTitor merged 2 commits intonotify-rs:mainfrom
stuhood:stuhood/dont-stat-for-non-recursive

Conversation

@stuhood
Copy link
Contributor

@stuhood stuhood commented Jul 14, 2020

Changing the order of the checks in EventLoop::add_watch avoids a stat for non-recursive watches.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@JohnTitor JohnTitor merged commit 05da071 into notify-rs:main Jul 16, 2020
@stuhood stuhood deleted the stuhood/dont-stat-for-non-recursive branch July 16, 2020 16:19
This was referenced Oct 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants