multi: fix lll linter errors

Author: yyforyongyu

rpc/rpcserver/server.go

@@ -188,7 +188,10 @@ func (s *walletServer) Accounts(ctx context.Context, req *pb.AccountsRequest) (
 func (s *walletServer) RenameAccount(ctx context.Context, req *pb.RenameAccountRequest) (
 	*pb.RenameAccountResponse, error) {
 
-	err := s.wallet.RenameAccountDeprecated(waddrmgr.KeyScopeBIP0044, req.GetAccountNumber(), req.GetNewName())
+	err := s.wallet.RenameAccountDeprecated(
+		waddrmgr.KeyScopeBIP0044, req.GetAccountNumber(),
+		req.GetNewName(),
+	)
 	if err != nil {
 		return nil, translateError(err)
 	}

waddrmgr/scoped_manager.go

@@ -1566,7 +1566,9 @@ func (s *ScopedKeyManager) CanAddAccount() error {
 // differs from the NewAccount method in that this method takes the account
 // number *directly*, rather than taking a string name for the account, then
 // mapping that to the next highest account number.
-func (s *ScopedKeyManager) NewRawAccount(ns walletdb.ReadWriteBucket, number uint32) error {
+func (s *ScopedKeyManager) NewRawAccount(
+	ns walletdb.ReadWriteBucket, number uint32) error {
+
 	s.mtx.Lock()
 	defer s.mtx.Unlock()
 

wallet/account_manager.go

@@ -196,8 +196,8 @@ func (w *Wallet) ListAccounts(_ context.Context) (*AccountsResult, error) {
 			return err
 		}
 
-		// Now, iterate through each key scope to assemble the final list
-		// of accounts with their properties and balances.
+		// Now, iterate through all key scopes to assemble the final
+		// list of accounts with their properties and balances.
 		for _, scopeMgr := range scopes {
 			scope := scopeMgr.Scope()
 			accountBalances := scopedBalances[scope]

wallet/account_manager_test.go

@@ -41,8 +41,8 @@ func TestNewAccount(t *testing.T) {
 	)
 	require.NoError(t, err, "unable to create new account")
 
-	// The new account should be the first account created, so it should have
-	// an index of 1.
+	// The new account should be the first account created, so it should
+	// have an index of 1.
 	require.Equal(t, uint32(1), account.AccountNumber, "expected account 1")
 
 	// We should be able to retrieve the account by its name.
@@ -251,7 +251,9 @@ func TestGetAccount(t *testing.T) {
 	require.NoError(t, err)
 
 	// We should be able to get the new account.
-	account, err := w.GetAccount(context.Background(), scope, testAccountName)
+	account, err := w.GetAccount(
+		context.Background(), scope, testAccountName,
+	)
 	require.NoError(t, err)
 	require.Equal(t, testAccountName, account.AccountName)
 	require.Equal(t, uint32(1), account.AccountNumber)
@@ -664,10 +666,13 @@ func TestFetchAccountBalances(t *testing.T) {
 		// Add UTXOs.
 		addTestUTXOForBalance(t, w, waddrmgr.KeyScopeBIP0084, 0, 100)
 		addTestUTXOForBalance(t, w, waddrmgr.KeyScopeBIP0084, 1, 200)
-		addTestUTXOForBalance(t, w, waddrmgr.KeyScopeBIP0049Plus, 1, 300)
+		addTestUTXOForBalance(
+			t, w, waddrmgr.KeyScopeBIP0049Plus, 1, 300,
+		)
 
 		// Update sync state.
-		err = walletdb.Update(w.db, func(tx walletdb.ReadWriteTx) error {
+		err = walletdb.Update(
+			w.db, func(tx walletdb.ReadWriteTx) error {
 			addrmgrNs := tx.ReadWriteBucket(waddrmgrNamespaceKey)
 			bs := &waddrmgr.BlockStamp{Height: 1}
 
@@ -733,7 +738,8 @@ func TestFetchAccountBalances(t *testing.T) {
 
 			var balances scopedBalances
 
-			err := walletdb.View(w.db, func(tx walletdb.ReadTx) error {
+			err := walletdb.View(
+				w.db, func(tx walletdb.ReadTx) error {
 				var err error
 
 				balances, err = w.fetchAccountBalances(

wallet/createtx.go

@@ -400,7 +400,9 @@ func (w *Wallet) findEligibleOutputs(dbtx walletdb.ReadTx,
 			continue
 		}
 
-		scopedMgr, addrAcct, err := w.addrStore.AddrAccount(addrmgrNs, addrs[0])
+		scopedMgr, addrAcct, err := w.addrStore.AddrAccount(
+			addrmgrNs, addrs[0],
+		)
 		if err != nil {
 			continue
 		}

wallet/import.go

@@ -189,7 +189,8 @@ func (w *Wallet) validateExtendedPubKey(pubKey *hdkeychain.ExtendedKey,
 	return nil
 }
 
-// ImportAccountDeprecated imports an account backed by an account extended public key.
+// ImportAccountDeprecated imports an account backed by an account extended
+// public key.
 // The master key fingerprint denotes the fingerprint of the root key
 // corresponding to the account public key (also known as the key with
 // derivation path m/). This may be required by some hardware wallets for proper
@@ -208,7 +209,8 @@ func (w *Wallet) validateExtendedPubKey(pubKey *hdkeychain.ExtendedKey,
 // distinction between the traditional BIP-0049 address schema (nested witness
 // pubkeys everywhere) and our own BIP-0049Plus address schema (nested
 // externally, witness internally).
-func (w *Wallet) ImportAccountDeprecated(name string, accountPubKey *hdkeychain.ExtendedKey,
+func (w *Wallet) ImportAccountDeprecated(
+	name string, accountPubKey *hdkeychain.ExtendedKey,
 	masterKeyFingerprint uint32, addrType *waddrmgr.AddressType) (
 	*waddrmgr.AccountProperties, error) {
 

wallet/interface.go

@@ -120,11 +120,14 @@ type Interface interface {
 	RenameAccountDeprecated(scope waddrmgr.KeyScope, account uint32,
 		newName string) error
 
-	// ImportAccountDeprecated imports an account backed by an extended public key.
+	// ImportAccountDeprecated imports an account backed by an extended
+	// public key.
+	//
 	// This creates a watch-only account.
 	//
 	// Deprecated: Use AccountManager.ImportAccount instead.
-	ImportAccountDeprecated(name string, accountPubKey *hdkeychain.ExtendedKey,
+	ImportAccountDeprecated(name string,
+		accountPubKey *hdkeychain.ExtendedKey,
 		masterKeyFingerprint uint32, addrType *waddrmgr.AddressType,
 	) (*waddrmgr.AccountProperties, error)
 

wallet/notifications.go

@@ -157,7 +157,9 @@ func makeTxSummary(dbtx walletdb.ReadTx, w *Wallet, details *wtxmgr.TxDetails) T
 func totalBalances(dbtx walletdb.ReadTx, w *Wallet, m map[uint32]btcutil.Amount) error {
 	addrmgrNs := dbtx.ReadBucket(waddrmgrNamespaceKey)
 
-	unspent, err := w.txStore.UnspentOutputs(dbtx.ReadBucket(wtxmgrNamespaceKey))
+	unspent, err := w.txStore.UnspentOutputs(
+		dbtx.ReadBucket(wtxmgrNamespaceKey),
+	)
 	if err != nil {
 		return err
 	}
@@ -167,7 +169,9 @@ func totalBalances(dbtx walletdb.ReadTx, w *Wallet, m map[uint32]btcutil.Amount)
 		_, addrs, _, err := txscript.ExtractPkScriptAddrs(
 			output.PkScript, w.chainParams)
 		if err == nil && len(addrs) > 0 {
-			_, outputAcct, err = w.addrStore.AddrAccount(addrmgrNs, addrs[0])
+			_, outputAcct, err = w.addrStore.AddrAccount(
+				addrmgrNs, addrs[0],
+			)
 		}
 		if err == nil {
 			_, ok := m[outputAcct]
@@ -234,7 +238,9 @@ func (s *NotificationServer) notifyUnminedTransaction(dbtx walletdb.ReadTx,
 
 	unminedTxs := []TransactionSummary{makeTxSummary(dbtx, s.wallet, details)}
 
-	unminedHashes, err := s.wallet.txStore.UnminedTxHashes(dbtx.ReadBucket(wtxmgrNamespaceKey))
+	unminedHashes, err := s.wallet.txStore.UnminedTxHashes(
+		dbtx.ReadBucket(wtxmgrNamespaceKey),
+	)
 	if err != nil {
 		log.Errorf("Cannot fetch unmined transaction hashes: %v", err)
 		return

wallet/utxos.go

@@ -71,7 +71,9 @@ func (w *Wallet) UnspentOutputs(policy OutputSelectionPolicy) ([]*TransactionOut
 				continue
 			}
 
-			_, outputAcct, err := w.addrStore.AddrAccount(addrmgrNs, addrs[0])
+			_, outputAcct, err := w.addrStore.AddrAccount(
+				addrmgrNs, addrs[0],
+			)
 			if err != nil {
 				return err
 			}