diff options
| author | Robin Jarry <robin@jarry.cc> | 2022-07-19 22:31:51 +0200 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-07-23 22:52:15 +0200 |
| commit | cd1999555714fb886493d2d04b6c472be55cebef (patch) | |
| tree | 1df3bcf5f687752db671d8bc9c7eab8a5c0fde71 /worker/imap/observer.go | |
| parent | a1f779ccc9b16b22ad6cb2e0bf73c290fd0cc756 (diff) | |
| download | aerc-cd1999555714fb886493d2d04b6c472be55cebef.tar.gz | |
logging: use level-based logger functions
Do not pass logger objects around anymore. Shuffle some messages to make
them consistent with the new logging API. Avoid using %v when a more
specific verb exists for the argument types.
The loggers are completely disabled (i.e. Sprintf is not even called)
by default. They are only enabled when redirecting stdout to a file.
Signed-off-by: Robin Jarry <robin@jarry.cc>
Acked-by: Moritz Poldrack <moritz@poldrack.dev>
Diffstat (limited to 'worker/imap/observer.go')
| -rw-r--r-- | worker/imap/observer.go | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/worker/imap/observer.go b/worker/imap/observer.go index e49744cc..448128c7 100644 --- a/worker/imap/observer.go +++ b/worker/imap/observer.go @@ -6,6 +6,7 @@ import ( "sync" "time" + "git.sr.ht/~rjarry/aerc/logging" "git.sr.ht/~rjarry/aerc/worker/types" "github.com/emersion/go-imap" ) @@ -146,7 +147,7 @@ func (o *observer) emit(errMsg string) { }, nil) } -func (o *observer) log(args ...interface{}) { - header := fmt.Sprintf("observer (%p) [running:%t]", o, o.running) - o.worker.Logger.Println(append([]interface{}{header}, args...)...) +func (o *observer) log(format string, args ...interface{}) { + msg := fmt.Sprintf(format, args...) + logging.Debugf("observer (%p) [running:%t] %s", o, o.running, msg) } |