diff options
author | Reto Brunner <reto@labrat.space> | 2020-05-27 07:52:13 +0200 |
---|---|---|
committer | Reto Brunner <reto@labrat.space> | 2020-05-27 08:11:40 +0200 |
commit | f06d683688e3d2139b14f67b7e349089e7200bf4 (patch) | |
tree | eee331112cbcfeafdb8cd6ba126d9e51ed72b10a /commands | |
parent | 0f78f06610c0e8887aba2ae50e99b86477a384b3 (diff) | |
download | aerc-f06d683688e3d2139b14f67b7e349089e7200bf4.tar.gz |
Remove duration from the status methods
We always set 10 seconds anyhow, might as well do that without repeating ourselfs.
Diffstat (limited to 'commands')
-rw-r--r-- | commands/account/mkdir.go | 5 | ||||
-rw-r--r-- | commands/account/view.go | 3 | ||||
-rw-r--r-- | commands/compose/attach.go | 9 | ||||
-rw-r--r-- | commands/compose/detach.go | 3 | ||||
-rw-r--r-- | commands/compose/postpone.go | 8 | ||||
-rw-r--r-- | commands/compose/send.go | 10 | ||||
-rw-r--r-- | commands/exec.go | 7 | ||||
-rw-r--r-- | commands/msg/archive.go | 5 | ||||
-rw-r--r-- | commands/msg/copy.go | 5 | ||||
-rw-r--r-- | commands/msg/delete.go | 7 | ||||
-rw-r--r-- | commands/msg/forward.go | 3 | ||||
-rw-r--r-- | commands/msg/modify-labels.go | 5 | ||||
-rw-r--r-- | commands/msg/move.go | 5 | ||||
-rw-r--r-- | commands/msg/pipe.go | 9 | ||||
-rw-r--r-- | commands/msg/read.go | 9 | ||||
-rw-r--r-- | commands/msg/recall.go | 3 | ||||
-rw-r--r-- | commands/msg/reply.go | 3 | ||||
-rw-r--r-- | commands/msgview/next.go | 4 | ||||
-rw-r--r-- | commands/msgview/open.go | 9 | ||||
-rw-r--r-- | commands/msgview/save.go | 4 | ||||
-rw-r--r-- | commands/pwd.go | 3 | ||||
-rw-r--r-- | commands/term.go | 3 | ||||
-rw-r--r-- | commands/util.go | 8 |
23 files changed, 54 insertions, 76 deletions
diff --git a/commands/account/mkdir.go b/commands/account/mkdir.go index f99fc01..fa0832d 100644 --- a/commands/account/mkdir.go +++ b/commands/account/mkdir.go @@ -3,7 +3,6 @@ package account import ( "errors" "strings" - "time" "git.sr.ht/~sircmpwn/aerc/widgets" "git.sr.ht/~sircmpwn/aerc/worker/types" @@ -37,10 +36,10 @@ func (MakeDir) Execute(aerc *widgets.Aerc, args []string) error { }, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Directory created.", 10*time.Second) + aerc.PushStatus("Directory created.") acct.Directories().Select(name) case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) return nil diff --git a/commands/account/view.go b/commands/account/view.go index d4653be..b421666 100644 --- a/commands/account/view.go +++ b/commands/account/view.go @@ -2,7 +2,6 @@ package account import ( "errors" - "time" "git.sr.ht/~sircmpwn/aerc/lib" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -42,7 +41,7 @@ func (ViewMessage) Execute(aerc *widgets.Aerc, args []string) error { lib.NewMessageStoreView(msg, store, aerc.DecryptKeys, func(view lib.MessageView, err error) { if err != nil { - aerc.PushError(err.Error(), 10*time.Second) + aerc.PushError(err.Error()) return } viewer := widgets.NewMessageViewer(acct, aerc.Config(), view) diff --git a/commands/compose/attach.go b/commands/compose/attach.go index 6b8d72f..148442b 100644 --- a/commands/compose/attach.go +++ b/commands/compose/attach.go @@ -4,7 +4,6 @@ import ( "fmt" "os" "strings" - "time" "git.sr.ht/~sircmpwn/aerc/commands" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -35,23 +34,23 @@ func (Attach) Execute(aerc *widgets.Aerc, args []string) error { path, err := homedir.Expand(path) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) return err } pathinfo, err := os.Stat(path) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) return err } else if pathinfo.IsDir() { - aerc.PushError("Attachment must be a file, not a directory", 10*time.Second) + aerc.PushError("Attachment must be a file, not a directory") return nil } composer, _ := aerc.SelectedTab().(*widgets.Composer) composer.AddAttachment(path) - aerc.PushSuccess(fmt.Sprintf("Attached %s", pathinfo.Name()), 10*time.Second) + aerc.PushSuccess(fmt.Sprintf("Attached %s", pathinfo.Name())) return nil } diff --git a/commands/compose/detach.go b/commands/compose/detach.go index 8bc0e88..b48159d 100644 --- a/commands/compose/detach.go +++ b/commands/compose/detach.go @@ -3,7 +3,6 @@ package compose import ( "fmt" "strings" - "time" "git.sr.ht/~sircmpwn/aerc/widgets" ) @@ -43,7 +42,7 @@ func (Detach) Execute(aerc *widgets.Aerc, args []string) error { return err } - aerc.PushSuccess(fmt.Sprintf("Detached %s", path), 10*time.Second) + aerc.PushSuccess(fmt.Sprintf("Detached %s", path)) return nil } diff --git a/commands/compose/postpone.go b/commands/compose/postpone.go index 90b6134..4427ff6 100644 --- a/commands/compose/postpone.go +++ b/commands/compose/postpone.go @@ -63,7 +63,7 @@ func (Postpone) Execute(aerc *widgets.Aerc, args []string) error { go func() { errStr := <-errChan if errStr != "" { - aerc.PushError(" "+errStr, 10*time.Second) + aerc.PushError(" " + errStr) return } @@ -71,7 +71,7 @@ func (Postpone) Execute(aerc *widgets.Aerc, args []string) error { ctr := datacounter.NewWriterCounter(ioutil.Discard) err = composer.WriteMessage(header, ctr) if err != nil { - aerc.PushError(errors.Wrap(err, "WriteMessage").Error(), 10*time.Second) + aerc.PushError(errors.Wrap(err, "WriteMessage").Error()) composer.Close() return } @@ -86,11 +86,11 @@ func (Postpone) Execute(aerc *widgets.Aerc, args []string) error { }, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Message postponed.", 10*time.Second) + aerc.PushStatus("Message postponed.") r.Close() composer.Close() case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) r.Close() composer.Close() } diff --git a/commands/compose/send.go b/commands/compose/send.go index a22be8f..e7ace85 100644 --- a/commands/compose/send.go +++ b/commands/compose/send.go @@ -221,14 +221,14 @@ func (Send) Execute(aerc *widgets.Aerc, args []string) error { } go func() { - aerc.PushStatus("Sending...", 10*time.Second) + aerc.PushStatus("Sending...") nbytes, err := sendAsync() if err != nil { aerc.SetError(" " + err.Error()) return } if config.CopyTo != "" { - aerc.PushStatus("Copying to "+config.CopyTo, 10*time.Second) + aerc.PushStatus("Copying to " + config.CopyTo) worker := composer.Worker() r, w := io.Pipe() worker.PostAction(&types.AppendMessage{ @@ -240,12 +240,12 @@ func (Send) Execute(aerc *widgets.Aerc, args []string) error { }, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Message sent.", 10*time.Second) + aerc.PushStatus("Message sent.") r.Close() composer.SetSent() composer.Close() case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) r.Close() composer.Close() } @@ -254,7 +254,7 @@ func (Send) Execute(aerc *widgets.Aerc, args []string) error { composer.WriteMessage(header, w) w.Close() } else { - aerc.PushStatus("Message sent.", 10*time.Second) + aerc.PushStatus("Message sent.") composer.SetSent() composer.Close() } diff --git a/commands/exec.go b/commands/exec.go index 0a5470d..f4a05a2 100644 --- a/commands/exec.go +++ b/commands/exec.go @@ -4,7 +4,6 @@ import ( "errors" "fmt" "os/exec" - "time" "git.sr.ht/~sircmpwn/aerc/widgets" ) @@ -31,16 +30,16 @@ func (ExecCmd) Execute(aerc *widgets.Aerc, args []string) error { go func() { err := cmd.Run() if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) } else { if cmd.ProcessState.ExitCode() != 0 { aerc.PushError(fmt.Sprintf( "%s: completed with status %d", args[0], - cmd.ProcessState.ExitCode()), 10*time.Second) + cmd.ProcessState.ExitCode())) } else { aerc.PushStatus(fmt.Sprintf( "%s: completed with status %d", args[0], - cmd.ProcessState.ExitCode()), 10*time.Second) + cmd.ProcessState.ExitCode())) } } }() diff --git a/commands/msg/archive.go b/commands/msg/archive.go index ba7e1f7..afd0d55 100644 --- a/commands/msg/archive.go +++ b/commands/msg/archive.go @@ -5,7 +5,6 @@ import ( "fmt" "path" "sync" - "time" "git.sr.ht/~sircmpwn/aerc/commands" "git.sr.ht/~sircmpwn/aerc/models" @@ -86,7 +85,7 @@ func (Archive) Execute(aerc *widgets.Aerc, args []string) error { case *types.Done: wg.Done() case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) success = false wg.Done() } @@ -96,7 +95,7 @@ func (Archive) Execute(aerc *widgets.Aerc, args []string) error { go func() { wg.Wait() if success { - aerc.PushStatus("Messages archived.", 10*time.Second) + aerc.PushStatus("Messages archived.") } }() return nil diff --git a/commands/msg/copy.go b/commands/msg/copy.go index e822c5c..9a8bcb8 100644 --- a/commands/msg/copy.go +++ b/commands/msg/copy.go @@ -3,7 +3,6 @@ package msg import ( "errors" "strings" - "time" "git.sr.ht/~sircmpwn/getopt" @@ -58,9 +57,9 @@ func (Copy) Execute(aerc *widgets.Aerc, args []string) error { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Messages copied.", 10*time.Second) + aerc.PushStatus("Messages copied.") case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) return nil diff --git a/commands/msg/delete.go b/commands/msg/delete.go index 482b60c..4663383 100644 --- a/commands/msg/delete.go +++ b/commands/msg/delete.go @@ -2,7 +2,6 @@ package msg import ( "errors" - "time" "git.sr.ht/~sircmpwn/aerc/lib" "git.sr.ht/~sircmpwn/aerc/models" @@ -45,9 +44,9 @@ func (Delete) Execute(aerc *widgets.Aerc, args []string) error { store.Delete(uids, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Messages deleted.", 10*time.Second) + aerc.PushStatus("Messages deleted.") case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) @@ -68,7 +67,7 @@ func (Delete) Execute(aerc *widgets.Aerc, args []string) error { lib.NewMessageStoreView(next, store, aerc.DecryptKeys, func(view lib.MessageView, err error) { if err != nil { - aerc.PushError(err.Error(), 10*time.Second) + aerc.PushError(err.Error()) return } nextMv := widgets.NewMessageViewer(acct, aerc.Config(), view) diff --git a/commands/msg/forward.go b/commands/msg/forward.go index 5dd51b2..28abbed 100644 --- a/commands/msg/forward.go +++ b/commands/msg/forward.go @@ -9,7 +9,6 @@ import ( "os" "path" "strings" - "time" "git.sr.ht/~sircmpwn/aerc/models" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -84,7 +83,7 @@ func (forward) Execute(aerc *widgets.Aerc, args []string) error { composer, err := widgets.NewComposer(aerc, acct, aerc.Config(), acct.AccountConfig(), acct.Worker(), template, defaults, original) if err != nil { - aerc.PushError("Error: "+err.Error(), 10*time.Second) + aerc.PushError("Error: " + err.Error()) return nil, err } diff --git a/commands/msg/modify-labels.go b/commands/msg/modify-labels.go index d74aece..02e6478 100644 --- a/commands/msg/modify-labels.go +++ b/commands/msg/modify-labels.go @@ -2,7 +2,6 @@ package msg import ( "errors" - "time" "git.sr.ht/~sircmpwn/aerc/commands" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -56,9 +55,9 @@ func (ModifyLabels) Execute(aerc *widgets.Aerc, args []string) error { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("labels updated", 10*time.Second) + aerc.PushStatus("labels updated") case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) return nil diff --git a/commands/msg/move.go b/commands/msg/move.go index a19194e..504beda 100644 --- a/commands/msg/move.go +++ b/commands/msg/move.go @@ -3,7 +3,6 @@ package msg import ( "errors" "strings" - "time" "git.sr.ht/~sircmpwn/getopt" @@ -69,9 +68,9 @@ func (Move) Execute(aerc *widgets.Aerc, args []string) error { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus("Message moved to "+joinedArgs, 10*time.Second) + aerc.PushStatus("Message moved to " + joinedArgs) case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) return nil diff --git a/commands/msg/pipe.go b/commands/msg/pipe.go index 15b8c52..15e0fda 100644 --- a/commands/msg/pipe.go +++ b/commands/msg/pipe.go @@ -5,7 +5,6 @@ import ( "fmt" "io" "os/exec" - "time" "git.sr.ht/~sircmpwn/aerc/commands" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -75,7 +74,7 @@ func (Pipe) Execute(aerc *widgets.Aerc, args []string) error { doTerm := func(reader io.Reader, name string) { term, err := commands.QuickTerm(aerc, cmd, reader) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) return } aerc.NewTab(term, name) @@ -93,16 +92,16 @@ func (Pipe) Execute(aerc *widgets.Aerc, args []string) error { }() err = ecmd.Run() if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) } else { if ecmd.ProcessState.ExitCode() != 0 { aerc.PushError(fmt.Sprintf( "%s: completed with status %d", cmd[0], - ecmd.ProcessState.ExitCode()), 10*time.Second) + ecmd.ProcessState.ExitCode())) } else { aerc.PushStatus(fmt.Sprintf( "%s: completed with status %d", cmd[0], - ecmd.ProcessState.ExitCode()), 10*time.Second) + ecmd.ProcessState.ExitCode())) } } } diff --git a/commands/msg/read.go b/commands/msg/read.go index e27f743..a08b807 100644 --- a/commands/msg/read.go +++ b/commands/msg/read.go @@ -3,7 +3,6 @@ package msg import ( "errors" "sync" - "time" "git.sr.ht/~sircmpwn/getopt" @@ -91,9 +90,9 @@ func submitReadChange(aerc *widgets.Aerc, store *lib.MessageStore, store.Read(uids, newState, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Done: - aerc.PushStatus(msg_success, 10*time.Second) + aerc.PushStatus(msg_success) case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) } }) } @@ -106,7 +105,7 @@ func submitReadChangeWg(aerc *widgets.Aerc, store *lib.MessageStore, case *types.Done: wg.Done() case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) *success = false wg.Done() } @@ -136,7 +135,7 @@ func submitToggle(aerc *widgets.Aerc, store *lib.MessageStore, h *helper) error go func() { wg.Wait() if success { - aerc.PushStatus(msg_success, 10*time.Second) + aerc.PushStatus(msg_success) } }() return nil diff --git a/commands/msg/recall.go b/commands/msg/recall.go index 6c5e973..7c9ac19 100644 --- a/commands/msg/recall.go +++ b/commands/msg/recall.go @@ -2,7 +2,6 @@ package msg import ( "io" - "time" "github.com/emersion/go-message" _ "github.com/emersion/go-message/charset" @@ -92,7 +91,7 @@ func (Recall) Execute(aerc *widgets.Aerc, args []string) error { }, func(msg types.WorkerMessage) { switch msg := msg.(type) { case *types.Error: - aerc.PushError(" "+msg.Error.Error(), 10*time.Second) + aerc.PushError(" " + msg.Error.Error()) composer.Close() } }) diff --git a/commands/msg/reply.go b/commands/msg/reply.go index 455c7ca..28ce245 100644 --- a/commands/msg/reply.go +++ b/commands/msg/reply.go @@ -7,7 +7,6 @@ import ( "io" gomail "net/mail" "strings" - "time" "git.sr.ht/~sircmpwn/getopt" @@ -140,7 +139,7 @@ func (reply) Execute(aerc *widgets.Aerc, args []string) error { composer, err := widgets.NewComposer(aerc, acct, aerc.Config(), acct.AccountConfig(), acct.Worker(), template, defaults, original) if err != nil { - aerc.PushError("Error: "+err.Error(), 10*time.Second) + aerc.PushError("Error: " + err.Error()) return err } diff --git a/commands/msgview/next.go b/commands/msgview/next.go index f9fb3d7..978cf10 100644 --- a/commands/msgview/next.go +++ b/commands/msgview/next.go @@ -1,8 +1,6 @@ package msgview import ( - "time" - "git.sr.ht/~sircmpwn/aerc/commands/account" "git.sr.ht/~sircmpwn/aerc/lib" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -42,7 +40,7 @@ func (NextPrevMsg) Execute(aerc *widgets.Aerc, args []string) error { lib.NewMessageStoreView(nextMsg, store, aerc.DecryptKeys, func(view lib.MessageView, err error) { if err != nil { - aerc.PushError(err.Error(), 10*time.Second) + aerc.PushError(err.Error()) return } nextMv := widgets.NewMessageViewer(acct, aerc.Config(), view) diff --git a/commands/msgview/open.go b/commands/msgview/open.go index 7f26542..e2545e3 100644 --- a/commands/msgview/open.go +++ b/commands/msgview/open.go @@ -7,7 +7,6 @@ import ( "io/ioutil" "mime" "os" - "time" "git.sr.ht/~sircmpwn/aerc/lib" "git.sr.ht/~sircmpwn/aerc/widgets" @@ -49,23 +48,23 @@ func (Open) Execute(aerc *widgets.Aerc, args []string) error { tmpFile, err := ioutil.TempFile(os.TempDir(), "aerc-*"+extension) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) return } defer tmpFile.Close() _, err = io.Copy(tmpFile, reader) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) return } err = lib.OpenFile(tmpFile.Name()) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) } - aerc.PushStatus("Opened", 10*time.Second) + aerc.PushStatus("Opened") }) return nil diff --git a/commands/msgview/save.go b/commands/msgview/save.go index ea1b8f3..7dbb0df 100644 --- a/commands/msgview/save.go +++ b/commands/msgview/save.go @@ -128,10 +128,10 @@ func (Save) Execute(aerc *widgets.Aerc, args []string) error { go func() { err := <-ch if err != nil { - aerc.PushError(fmt.Sprintf("Save failed: %v", err), 10*time.Second) + aerc.PushError(fmt.Sprintf("Save failed: %v", err)) return } - aerc.PushStatus("Saved to "+path, 10*time.Second) + aerc.PushStatus("Saved to " + path) }() return nil } diff --git a/commands/pwd.go b/commands/pwd.go index d3f0e0c..fc9a411 100644 --- a/commands/pwd.go +++ b/commands/pwd.go @@ -3,7 +3,6 @@ package commands import ( "errors" "os" - "time" "git.sr.ht/~sircmpwn/aerc/widgets" ) @@ -30,6 +29,6 @@ func (PrintWorkDir) Execute(aerc *widgets.Aerc, args []string) error { if err != nil { return err } - aerc.PushStatus(pwd, 10*time.Second) + aerc.PushStatus(pwd) return nil } diff --git a/commands/term.go b/commands/term.go index 9023285..00f6937 100644 --- a/commands/term.go +++ b/commands/term.go @@ -2,7 +2,6 @@ package commands import ( "os/exec" - "time" "github.com/riywo/loginshell" @@ -47,7 +46,7 @@ func TermCore(aerc *widgets.Aerc, args []string) error { term.OnClose = func(err error) { aerc.RemoveTab(term) if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) } } return nil diff --git a/commands/util.go b/commands/util.go index 7c7b6ab..bab9774 100644 --- a/commands/util.go +++ b/commands/util.go @@ -8,7 +8,6 @@ import ( "path/filepath" "sort" "strings" - "time" "git.sr.ht/~sircmpwn/aerc/lib" "git.sr.ht/~sircmpwn/aerc/models" @@ -32,12 +31,11 @@ func QuickTerm(aerc *widgets.Aerc, args []string, stdin io.Reader) (*widgets.Ter term.OnClose = func(err error) { if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) // remove the tab on error, otherwise it gets stuck aerc.RemoveTab(term) } else { - aerc.PushStatus("Process complete, press any key to close.", - 10*time.Second) + aerc.PushStatus("Process complete, press any key to close.") term.OnEvent = func(event tcell.Event) bool { aerc.RemoveTab(term) return true @@ -56,7 +54,7 @@ func QuickTerm(aerc *widgets.Aerc, args []string, stdin io.Reader) (*widgets.Ter err := <-status if err != nil { - aerc.PushError(" "+err.Error(), 10*time.Second) + aerc.PushError(" " + err.Error()) } } |