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/authinfo.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/authinfo.go')
| -rw-r--r-- | widgets/authinfo.go | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/widgets/authinfo.go b/widgets/authinfo.go index e773240e..a57d8a13 100644 --- a/widgets/authinfo.go +++ b/widgets/authinfo.go @@ -25,20 +25,17 @@ func (a *AuthInfo) Draw(ctx *ui.Context) { defaultStyle := a.uiConfig.GetStyle(config.STYLE_DEFAULT) ctx.Fill(0, 0, ctx.Width(), ctx.Height(), ' ', defaultStyle) var text string - if a.authdetails == nil { + switch { + case a.authdetails == nil: text = "(no header)" ctx.Printf(0, 0, defaultStyle, text) - } else if a.authdetails.Err != nil { + case a.authdetails.Err != nil: style := a.uiConfig.GetStyle(config.STYLE_ERROR) text = a.authdetails.Err.Error() ctx.Printf(0, 0, style, text) - } else { + default: checkBounds := func(x int) bool { - if x < ctx.Width() { - return true - } else { - return false - } + return x < ctx.Width() } setResult := func(result auth.Result) (string, tcell.Style) { switch result { |