Small correctness and formatting fixes.

Sircular · Sircular · commit ebbcb9de2d62 · 2020-07-07T16:28:27.000-05:00
* Use `defer` for WaitGroup in case of panic.
* Fix wording in comment.

Signed-off-by: Walter Mays &lt;walt@waltermays.com&gt;
diff --git a/collector/filesystem_linux.go b/collector/filesystem_linux.go
@@ -65,10 +65,10 @@ func (c *filesystemCollector) GetStats() ([]filesystemStats, error) {
 	for i := 0; i < workerCount; i++ {
 		wg.Add(1)
 		go func() {
+			defer wg.Done()
 			for labels := range labelChan {
 				statChan <- c.processStat(labels)
 			}
-			wg.Done()
 		}()
 	}
 
@@ -107,7 +107,7 @@ func (c *filesystemCollector) GetStats() ([]filesystemStats, error) {
 }
 
 func (c *filesystemCollector) processStat(labels filesystemLabels) filesystemStats {
-	// The success channel is used do tell the "watcher" that the stat
+	// The success channel is used to tell the "watcher" that the stat
 	// finished successfully. The channel is closed on success.
 	success := make(chan struct{})
 	go stuckMountWatcher(labels.mountPoint, success, c.logger)

Original file line number	Diff line number	Diff line change
`@@ -65,10 +65,10 @@ func (c *filesystemCollector) GetStats() ([]filesystemStats, error) {`
`65`	`65`	`for i := 0; i < workerCount; i++ {`
`66`	`66`	`wg.Add(1)`
`67`	`67`	`go func() {`
	`68`	`+ defer wg.Done()`
`68`	`69`	`for labels := range labelChan {`
`69`	`70`	`statChan <- c.processStat(labels)`
`70`	`71`	`}`
`71`		`- wg.Done()`
`72`	`72`	`}()`
`73`	`73`	`}`
`74`	`74`
`@@ -107,7 +107,7 @@ func (c *filesystemCollector) GetStats() ([]filesystemStats, error) {`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`func (c *filesystemCollector) processStat(labels filesystemLabels) filesystemStats {`
`110`		`- // The success channel is used do tell the "watcher" that the stat`
	`110`	`+ // The success channel is used to tell the "watcher" that the stat`
`111`	`111`	`// finished successfully. The channel is closed on success.`
`112`	`112`	`success := make(chan struct{})`
`113`	`113`	`go stuckMountWatcher(labels.mountPoint, success, c.logger)`