-
Notifications
You must be signed in to change notification settings - Fork 207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
implement passive debouncer without blocking #467
Conversation
c1feee8
to
1ce7060
Compare
cc @kchibisov |
1ce7060
to
1cb521d
Compare
notify-debouncer-mini/src/lib.rs
Outdated
} | ||
} | ||
|
||
// // drain events channel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if we want this, it certainly allows us to receive rapid fired events.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that it'll provide any value? The "rapid fire" events will be handled by recv_timeout
anyway just fine.
551dc86
to
0de9a72
Compare
notify-debouncer-mini/src/lib.rs
Outdated
} | ||
} | ||
|
||
// // drain events channel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that it'll provide any value? The "rapid fire" events will be handled by recv_timeout
anyway just fine.
5051444
to
bacc7c5
Compare
1febce8
to
08e685c
Compare
ca4b2c3
to
9dd1e8c
Compare
Switches from active polling to an event loop that can fully sleep. Also overhauls the configuration with a builder. Bumps version to 0.4
9dd1e8c
to
96ea260
Compare
CC @dfaust because we'll have to retrofit this for the debouncer-full too (especially the semver breaking config change, to future proof additional config values) |
Remove active polling and introduce a passive waiting method that can also shutdown everything.
This fixes #466