Skip to content
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.

Sign up for GitHub

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

Some fixes regarding unwatching and rewatching a path #129

Merged
merged 14 commits into from
Oct 4, 2017
Merged

Some fixes regarding unwatching and rewatching a path #129

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
ca15b52
tests: pass root dir of testing watcher through cleanpath
imsodin Aug 29, 2017
7321520
trigger: Check that no watches remain on close
imsodin Aug 31, 2017
07c566e
Add tests for unwatching and then watching the same path and set test…
imsodin Aug 29, 2017
cea90c3
readdcw: check state on every loop iteration
imsodin Aug 30, 2017
de41129
readdcw: Fix race (missing lock)
imsodin Aug 29, 2017
1687b00
readdcw: Manually send unwatch events when watched file/folder was re…
imsodin Aug 30, 2017
40b317d
fsevents: Don't canonicalize paths, tree does it already
imsodin Aug 28, 2017
734d890
inotify: Handle removing of watches on non-existing paths
imsodin Aug 28, 2017
9c33e90
trigger: When unwatching, rewatch in the correct mode (dir/ndir)
imsodin Aug 31, 2017
a55fe76
trigger: typo when watching fails (return nil -> err)
imsodin Aug 31, 2017
e2ec277
trigger: Specify common watched structure in watcher_trigger.go (dedup)
imsodin Aug 31, 2017
46fab92
readdcw: polish (defer r.Unlock() where appropriate)
imsodin Aug 30, 2017
a65f5f9
readdcw: Some useful debug logging
imsodin Aug 30, 2017
9a90a47
readdcw: Don't use atomic when lock is held
imsodin Oct 4, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
trigger: Check that no watches remain on close
  • Loading branch information
@imsodin
imsodin committed Aug 31, 2017
commit 732152040a151848b3d92e6b25d41b7cf7db9ec3
4 changes: 4 additions & 0 deletions watcher_trigger.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
package notify

import (
"fmt"
"os"
"path/filepath"
"strings"
Expand Down Expand Up @@ -117,6 +118,9 @@ func (t *trg) Close() (err error) {
dbgprintf("trg: closing native watch failed: %q\n", e)
err = nonil(err, e)
}
if remaining := len(t.pthLkp); remaining != 0 {
err = nonil(err, fmt.Errorf("Not all watches were removed: len(t.pthLkp) == %v", len(t.pthLkp)))
}
t.Unlock()
return
}
Expand Down