Move Unlock under defer

cmaglie · cmaglie · commit fc8501c6320e · 2022-07-20T16:41:44.000+02:00
diff --git a/arduino/discovery/discoverymanager/discoverymanager.go b/arduino/discovery/discoverymanager/discoverymanager.go
@@ -150,9 +150,9 @@ func (dm *DiscoveryManager) Watch() (*PortWatcher, error) {
 	}
 	watcher.closeCB = func() {
 		dm.watchersMutex.Lock()
+		defer dm.watchersMutex.Unlock()
 		delete(dm.watchers, watcher)
 		close(watcher.feed)
-		dm.watchersMutex.Unlock()
 	}
 	go func() {
 		dm.watchersMutex.Lock()

Original file line number	Diff line number	Diff line change
`@@ -150,9 +150,9 @@ func (dm DiscoveryManager) Watch() (PortWatcher, error) {`
`150`	`150`	`}`
`151`	`151`	`watcher.closeCB = func() {`
`152`	`152`	`dm.watchersMutex.Lock()`
	`153`	`+ defer dm.watchersMutex.Unlock()`
`153`	`154`	`delete(dm.watchers, watcher)`
`154`	`155`	`close(watcher.feed)`
`155`		`- dm.watchersMutex.Unlock()`
`156`	`156`	`}`
`157`	`157`	`go func() {`
`158`	`158`	`dm.watchersMutex.Lock()`