Update todos

cmd/gotop/main.go

@@ -334,17 +334,7 @@ func eventLoop(c gotop.Config, grid *layout.MyGrid) {
 	}
 }
 
-// TODO: @devices fans
-// TODO: @devices mpd visualizer
-// TODO: @devices color bars for memory, a-la bashtop
-// TODO: Add tab completion for Linux https://gist.github.com/icholy/5314423
 // TODO: state:merge #135 linux console font (cmatsuoka/console-font)
-// TODO: Abstract out the UI toolkit.  mum4k/termdash, VladimirMarkelov/clui, gcla/gowid, rivo/tview, marcusolsson/tui-go might work better for some OS/Archs. Performance/memory use comparison would be interesting.
-// TODO: more unit tests, benchmarks
-// TODO: README is getting long. Move to wiki.
-// TODO: add verbose debugging option
-// TODO: find VMs for FreeBSD, etc for testing gotop
-// TODO: add README about extensions, and wiki page for writing extensions
 func main() {
 	// For performance testing
 	//go func() {

devices/cpu.go

@@ -1,7 +1,5 @@
 package devices
 
-// TODO: https://github.com/elastic/go-sysinfo
-
 import (
 	"log"
 	"time"

devices/devices.go

@@ -8,6 +8,8 @@ const (
 	Temperatures = "Temperatures" // Device domain for temperature sensors
 )
 
+// TODO: Redesign; this is not thread safe, and it's easy to write code that triggers concurrent modification panics. Channels?
+
 var Domains []string = []string{Temperatures}
 var _shutdownFuncs []func() error
 var _devs map[string][]string

devices/temp_openbsd.go

@@ -15,7 +15,7 @@ import (
 	"unsafe"
 )
 
-// TODO: Add filtering.  Getting the temperature sensor names is non-trivial for OpenBSD, and until I can test it, leave it unimplemented
+// TODO: Add sensor filtering
 func init() {
 	RegisterTemp(update)
 }

widgets/batterygauge.go

@@ -57,6 +57,7 @@ func (b *BatteryGauge) EnableMetric() {
 }
 
 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)

widgets/disk.go

@@ -32,8 +32,6 @@ type DiskWidget struct {
 	metric         map[string]prometheus.Gauge
 }
 
-// TODO: Add filtering
-// TODO: Abstract out device from widget
 func NewDiskWidget() *DiskWidget {
 	self := &DiskWidget{
 		Table:          ui.NewTable(),