diff options
| author | Koni Marti <koni.marti@gmail.com> | 2022-10-02 11:36:05 +0200 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-10-02 18:56:26 +0200 |
| commit | dc299cc8adda5c9fb7a0f6a7719f3e2cd0f1f443 (patch) | |
| tree | 5a3e0ded9d72ae0eafa3440e2ee1c88109d9c21a /widgets/terminal.go | |
| parent | 020279108c4234974a26580183ef60945c5bd5cb (diff) | |
| download | aerc-dc299cc8adda5c9fb7a0f6a7719f3e2cd0f1f443.tar.gz | |
logging: substitute %w for %v
Subsitute the format specifier %w for %v in the logging facility. The
logging functions use a fmt.Sprintf call behind the scene which does not
recognize %w. %w should be used in fmt.Errorf when you want to wrap
errors. Hence, the log entries that use %w are improperly formatted like
this:
ERROR 2022/10/02 09:13:57.724529 worker.go:439: could not get message
info %!w(*fmt.wrapError=&{could not get structure: [snip] })
^
Links: https://go.dev/blog/go1.13-errors
Signed-off-by: Koni Marti <koni.marti@gmail.com>
Acked-by: Moritz Poldrack <moritz@poldrack.dev>
Diffstat (limited to 'widgets/terminal.go')
| -rw-r--r-- | widgets/terminal.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/widgets/terminal.go b/widgets/terminal.go index 894bdf09..b854a288 100644 --- a/widgets/terminal.go +++ b/widgets/terminal.go @@ -84,7 +84,7 @@ func (term *Terminal) Draw(ctx *ui.Context) { term.vterm.Watch(term) attr := &syscall.SysProcAttr{Setsid: true, Setctty: true, Ctty: 1} if err := term.vterm.StartWithAttrs(term.cmd, attr); err != nil { - logging.Errorf("error running terminal: %w", err) + logging.Errorf("error running terminal: %v", err) term.Close(err) return } |