meutel/oauth2_proxy
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

watcher: properly break out in nested blocks

Browse Source 
Found via staticcheck:

watcher.go:48:5: ineffective break statement. Did you mean to break out of the outer loop? (SA4011)
This commit is contained in:
Adam Shannon 2019-01-30 18:54:27 -06:00
parent 090ff11923
commit 6a775b97c9
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
watcher.go
View File

@ -45,7 +45,7 @@ func WatchForUpdates(filename string, done <-chan bool, action func()) {
select { select {
case _ = <-done: case _ = <-done:
log.Printf("Shutting down watcher for: %s", filename) log.Printf("Shutting down watcher for: %s", filename)
break return
case event := <-watcher.Events: case event := <-watcher.Events:
// On Arch Linux, it appears Chmod events precede Remove events, // On Arch Linux, it appears Chmod events precede Remove events,
// which causes a race between action() and the coming Remove event. // which causes a race between action() and the coming Remove event.