diff options
author | Moritz Poldrack <git@moritz.sh> | 2022-07-31 14:32:48 +0200 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2022-08-04 21:58:01 +0200 |
commit | 978d35d356e8752bdd272884df48a6289d88b40a (patch) | |
tree | 3910243e688ef503159d07ce44b22cfea5d6c6fd /widgets/dirlist.go | |
parent | c882cf9960be691fe55617b87cdfcfbabd5d5557 (diff) | |
download | aerc-978d35d356e8752bdd272884df48a6289d88b40a.tar.gz |
lint: homogenize operations and minor fixes (gocritic)
Apply GoDoc comment policy (comments for humans should have a space
after the //; machine-readable comments shouldn't)
Use strings.ReplaceAll instead of strings.Replace when appropriate
Remove if/else chains by replacing them with switches
Use short assignment/increment notation
Replace single case switches with if statements
Combine else and if when appropriate
Signed-off-by: Moritz Poldrack <moritz@poldrack.dev>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'widgets/dirlist.go')
-rw-r--r-- | widgets/dirlist.go | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/widgets/dirlist.go b/widgets/dirlist.go index df4e8192..03f9239a 100644 --- a/widgets/dirlist.go +++ b/widgets/dirlist.go @@ -270,11 +270,12 @@ func (dirlist *DirectoryList) getRUEString(name string) string { } di := msgStore.DirInfo rueString := "" - if di.Recent > 0 { + switch { + case di.Recent > 0: rueString = fmt.Sprintf("%d/%d/%d", di.Recent, di.Unseen, di.Exists) - } else if di.Unseen > 0 { + case di.Unseen > 0: rueString = fmt.Sprintf("%d/%d", di.Unseen, di.Exists) - } else if di.Exists > 0 { + case di.Exists > 0: rueString = fmt.Sprintf("%d", di.Exists) } return rueString @@ -358,8 +359,7 @@ func (dirlist *DirectoryList) drawScrollbar(ctx *ui.Context) { } func (dirlist *DirectoryList) MouseEvent(localX int, localY int, event tcell.Event) { - switch event := event.(type) { - case *tcell.EventMouse: + if event, ok := event.(*tcell.EventMouse); ok { switch event.Buttons() { case tcell.Button1: clickedDir, ok := dirlist.Clicked(localX, localY) |