Removes unnecessary return value

Author: bndw

export.go

@@ -12,21 +12,12 @@ func exportCommand(args ...string) int {
 		return handleError(err)
 	}
 
-	accounts, err := safe.List()
-	if err != nil {
-		return handleError(err)
-	}
-
+	accounts := safe.List()
 	if len(accounts) < 1 {
 		fmt.Println("No accounts to export")
 		return 1
 	}
 
-	var accountSlice []interface{}
-	for _, account := range accounts {
-		accountSlice = append(accountSlice, account)
-	}
-
-	utils.PrettyPrint(accountSlice)
+	utils.PrettyPrint(accounts)
 	return 0
 }

list.go

@@ -11,13 +11,8 @@ func listCommand(args ...string) int {
 		return handleError(err)
 	}
 
-	accounts, err := safe.List()
-	if err != nil {
-		return handleError(err)
-	}
-
 	var accountNames []string
-	for _, account := range accounts {
+	for _, account := range safe.List() {
 		accountNames = append(accountNames, account.Name)
 	}
 

safe/list.go

@@ -1,10 +1,10 @@
 package safe
 
-func (s *Safe) List() ([]Account, error) {
+func (s *Safe) List() []Account {
 	var accounts []Account
 	for _, account := range s.Accounts {
 		accounts = append(accounts, account)
 	}
 
-	return accounts, nil
+	return accounts
 }

safe/list_test.go

@@ -16,11 +16,7 @@ func TestList(t *testing.T) {
 		t.Error(err)
 	}
 
-	accounts, err := safe.List()
-	if err != nil {
-		t.Error(err)
-	}
-
+	accounts := safe.List()
 	if len(accounts) < 1 {
 		t.Error(err)
 	}