aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDrew DeVault <sir@cmpwn.com>2018-02-01 18:59:13 -0500
committerDrew DeVault <sir@cmpwn.com>2018-02-01 18:59:13 -0500
commit1767e4fab515cefba08345e6eb2c84f46daf4be1 (patch)
tree562b1f2269df58b4c783a771ac0324ce32ec7bb1
parentd24e4712a45e03d79fa1ccb71d00c5b830d5a305 (diff)
downloadaerc-1767e4fab515cefba08345e6eb2c84f46daf4be1.tar.gz
Improve logging
-rw-r--r--cmd/aerc/main.go6
-rw-r--r--worker/types/worker.go26
2 files changed, 22 insertions, 10 deletions
diff --git a/cmd/aerc/main.go b/cmd/aerc/main.go
index cda49911..42199789 100644
--- a/cmd/aerc/main.go
+++ b/cmd/aerc/main.go
@@ -22,7 +22,7 @@ func main() {
} else {
logOut = ioutil.Discard
}
- logger = log.New(logOut, "", log.LstdFlags|log.Lshortfile)
+ logger = log.New(logOut, "", log.LstdFlags)
logger.Println("Starting up aerc")
conf, err := config.LoadConfig(nil)
@@ -37,9 +37,7 @@ func main() {
for _, account := range conf.Accounts {
logger.Printf("Initializing account %s\n", account.Name)
tab, err := ui.NewAccountTab(&account, log.New(
- logOut,
- fmt.Sprintf("[%s] ", account.Name),
- log.LstdFlags|log.Lshortfile))
+ logOut, fmt.Sprintf("[%s] ", account.Name), log.LstdFlags))
if err != nil {
panic(err)
}
diff --git a/worker/types/worker.go b/worker/types/worker.go
index a99d4326..1ff16a4c 100644
--- a/worker/types/worker.go
+++ b/worker/types/worker.go
@@ -19,7 +19,11 @@ type Worker struct {
func (worker *Worker) PostAction(msg WorkerMessage,
cb func(msg WorkerMessage)) {
- worker.Logger.Printf("=> %T\n", msg)
+ if resp := msg.InResponseTo(); resp != nil {
+ worker.Logger.Printf("(ui)=> %T:%T\n", msg, resp)
+ } else {
+ worker.Logger.Printf("(ui)=> %T\n", msg)
+ }
worker.Actions <- msg
if cb != nil {
@@ -30,7 +34,11 @@ func (worker *Worker) PostAction(msg WorkerMessage,
func (worker *Worker) PostMessage(msg WorkerMessage,
cb func(msg WorkerMessage)) {
- worker.Logger.Printf("-> %T\n", msg)
+ if resp := msg.InResponseTo(); resp != nil {
+ worker.Logger.Printf("->(ui) %T:%T\n", msg, resp)
+ } else {
+ worker.Logger.Printf("->(ui) %T\n", msg)
+ }
worker.Messages <- msg
if cb != nil {
@@ -39,8 +47,11 @@ func (worker *Worker) PostMessage(msg WorkerMessage,
}
func (worker *Worker) ProcessMessage(msg WorkerMessage) WorkerMessage {
-
- worker.Logger.Printf("<= %T\n", msg)
+ if resp := msg.InResponseTo(); resp != nil {
+ worker.Logger.Printf("(ui)<= %T:%T\n", msg, resp)
+ } else {
+ worker.Logger.Printf("(ui)<= %T\n", msg)
+ }
if cb, ok := worker.Callbacks[msg.InResponseTo()]; ok {
cb(msg)
delete(worker.Callbacks, msg)
@@ -49,8 +60,11 @@ func (worker *Worker) ProcessMessage(msg WorkerMessage) WorkerMessage {
}
func (worker *Worker) ProcessAction(msg WorkerMessage) WorkerMessage {
-
- worker.Logger.Printf("<- %T\n", msg)
+ if resp := msg.InResponseTo(); resp != nil {
+ worker.Logger.Printf("<-(ui) %T:%T\n", msg, resp)
+ } else {
+ worker.Logger.Printf("<-(ui) %T\n", msg)
+ }
if cb, ok := worker.Callbacks[msg.InResponseTo()]; ok {
cb(msg)
delete(worker.Callbacks, msg)