diff options
| author | Robin Jarry <robin@jarry.cc> | 2023-10-10 00:08:31 +0200 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2023-10-10 11:38:01 +0200 |
| commit | bc176bd61ba726351a489cabf4da16a47dc5ec3b (patch) | |
| tree | bbf06f731592d072f3d6f76f1648d61989375f2e /commands/quit.go | |
| parent | 598e4a5803578ab3e291f232d6aad31b4efd8ea4 (diff) | |
| download | aerc-bc176bd61ba726351a489cabf4da16a47dc5ec3b.tar.gz | |
app: export global functions
The single Aerc object is passed around in almost all command functions.
This hinders readability.
Store the single Aerc instance as a global variable. Export public
functions from the app package to access methods of that object. Remove
all explicit references to *app.Aerc and replace them with calls to
these functions. For references to private/unexported fields and
functions from within the app package, directly access the global aerc
object.
Signed-off-by: Robin Jarry <robin@jarry.cc>
Acked-by: Moritz Poldrack <moritz@poldrack.dev>
Diffstat (limited to 'commands/quit.go')
| -rw-r--r-- | commands/quit.go | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/commands/quit.go b/commands/quit.go index 73054f86..42e8fc95 100644 --- a/commands/quit.go +++ b/commands/quit.go @@ -4,7 +4,6 @@ import ( "errors" "fmt" - "git.sr.ht/~rjarry/aerc/app" "git.sr.ht/~rjarry/aerc/commands/mode" "git.sr.ht/~sircmpwn/getopt" ) @@ -19,7 +18,7 @@ func (Quit) Aliases() []string { return []string{"quit", "exit"} } -func (Quit) Complete(aerc *app.Aerc, args []string) []string { +func (Quit) Complete(args []string) []string { return nil } @@ -29,7 +28,7 @@ func (err ErrorExit) Error() string { return "exit" } -func (Quit) Execute(aerc *app.Aerc, args []string) error { +func (Quit) Execute(args []string) error { force := false opts, optind, err := getopt.Getopts(args, "f") if err != nil { |