diff options
| author | Sean E. Russell <ser@ser1.net> | 2020-05-31 14:45:24 -0500 |
|---|---|---|
| committer | Sean E. Russell <ser@ser1.net> | 2020-05-31 14:45:24 -0500 |
| commit | 5123568b923991f9fba318954ea3e8cf6785e252 (patch) | |
| tree | be243db1693d3a25419e03369f9f922c01d2f31a | |
| parent | e8727c7c201f629126b968373e59183325915be5 (diff) | |
Only report battery sensor errors once; closes #117
| -rw-r--r-- | CHANGELOG.md | 1 | ||||
| -rw-r--r-- | widgets/batterygauge.go | 9 |
2 files changed, 8 insertions, 2 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index 53b5a2c..b5ae31c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -61,6 +61,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - There were a number of minor issues with the config file that have been cleaned up. - Compile errors on FreeBSD due to golang.org/x/sys API breakages - Key bindings now work in FreeBSD (#95) +- Only report battery sensor errors once (reduce noise in the log, #117) ## [3.5.3] - 2020-05-30 diff --git a/widgets/batterygauge.go b/widgets/batterygauge.go index f28dd34..135f223 100644 --- a/widgets/batterygauge.go +++ b/widgets/batterygauge.go @@ -39,11 +39,16 @@ func (b *BatteryGauge) EnableMetric() { }) } +// Only report battery errors once. +var errLogged = false + func (b *BatteryGauge) update() { - // FIXME: Getting a lot of these in the logs bats, err := battery.GetAll() if err != nil { - log.Printf("error setting up batteries: %v", err) + if !errLogged { + log.Printf("error setting up batteries: %v", err) + errLogged = true + } return } mx := 0.0 |