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>

1 files changed, 4 insertions, 4 deletions

diff --git a/commands/new-account.go b/commands/new-account.go
index 3170c75c..0f84141f 100644
--- a/commands/new-account.go
+++ b/commands/new-account.go
@@ -17,22 +17,22 @@ func (NewAccount) Aliases() []string {
 	return []string{"new-account"}
 }
 
-func (NewAccount) Complete(aerc *app.Aerc, args []string) []string {
+func (NewAccount) Complete(args []string) []string {
 	return nil
 }
 
-func (NewAccount) Execute(aerc *app.Aerc, args []string) error {
+func (NewAccount) Execute(args []string) error {
 	opts, _, err := getopt.Getopts(args, "t")
 	if err != nil {
 		return errors.New("Usage: new-account [-t]")
 	}
-	wizard := app.NewAccountWizard(aerc)
+	wizard := app.NewAccountWizard()
 	for _, opt := range opts {
 		if opt.Option == 't' {
 			wizard.ConfigureTemporaryAccount(true)
 		}
 	}
 	wizard.Focus(true)
-	aerc.NewTab(wizard, "New account")
+	app.NewTab(wizard, "New account")
 	return nil
 }