multi: fix formatting issues proposed by make fmt

Author: mohamedawnallah

internal/legacy/keystore/keystore.go

@@ -21,9 +21,6 @@ import (
 	"sync"
 	"time"
 
-	// consider vendoring this deprecated ripemd160 package
-	"golang.org/x/crypto/ripemd160" // nolint:staticcheck
-
 	"github.com/btcsuite/btcd/btcec/v2"
 	"github.com/btcsuite/btcd/btcec/v2/ecdsa"
 	"github.com/btcsuite/btcd/btcutil"
@@ -33,6 +30,7 @@ import (
 	"github.com/btcsuite/btcd/wire"
 	"github.com/btcsuite/btcwallet/internal/legacy/rename"
 	secp "github.com/decred/dcrd/dcrec/secp256k1/v4"
+	"golang.org/x/crypto/ripemd160" // nolint:staticcheck
 )
 
 const (

rpc/legacyrpc/server.go

@@ -164,7 +164,7 @@ func NewServer(opts *Options, walletLoader *wallet.Loader, listeners []net.Liste
 // httpBasicAuth returns the UTF-8 bytes of the HTTP Basic authentication
 // string:
 //
-//   "Basic " + base64(username + ":" + password)
+//	"Basic " + base64(username + ":" + password)
 func httpBasicAuth(username, password string) []byte {
 	const header = "Basic "
 	base64 := base64.StdEncoding

rpc/rpcserver/log.go

@@ -18,9 +18,8 @@ import (
 	"os"
 	"strings"
 
-	"google.golang.org/grpc/grpclog"
-
 	"github.com/btcsuite/btclog"
+	"google.golang.org/grpc/grpclog"
 )
 
 // UseLogger sets the logger to use for the gRPC server.

rpc/rpcserver/server.go

@@ -8,11 +8,11 @@
 // Full documentation of the API implemented by this package is maintained in a
 // language-agnostic document:
 //
-//   https://github.com/btcsuite/btcwallet/blob/master/rpc/documentation/api.md
+//	https://github.com/btcsuite/btcwallet/blob/master/rpc/documentation/api.md
 //
 // Any API changes must be performed according to the steps listed here:
 //
-//   https://github.com/btcsuite/btcwallet/blob/master/rpc/documentation/serverchanges.md
+//	https://github.com/btcsuite/btcwallet/blob/master/rpc/documentation/serverchanges.md
 package rpcserver
 
 import (
@@ -21,11 +21,6 @@ import (
 	"sync"
 	"time"
 
-	"golang.org/x/net/context"
-	"google.golang.org/grpc"
-	"google.golang.org/grpc/codes"
-	"google.golang.org/grpc/status"
-
 	"github.com/btcsuite/btcd/btcutil"
 	"github.com/btcsuite/btcd/btcutil/hdkeychain"
 	"github.com/btcsuite/btcd/chaincfg/chainhash"
@@ -40,6 +35,10 @@ import (
 	"github.com/btcsuite/btcwallet/waddrmgr"
 	"github.com/btcsuite/btcwallet/wallet"
 	"github.com/btcsuite/btcwallet/walletdb"
+	"golang.org/x/net/context"
+	"google.golang.org/grpc"
+	"google.golang.org/grpc/codes"
+	"google.golang.org/grpc/status"
 )
 
 // Public API version constants
@@ -486,12 +485,12 @@ func (s *walletServer) SignTransaction(ctx context.Context, req *pb.SignTransact
 }
 
 // BUGS:
-// - The transaction is not inspected to be relevant before publishing using
-//   sendrawtransaction, so connection errors to btcd could result in the tx
-//   never being added to the wallet database.
-// - Once the above bug is fixed, wallet will require a way to purge invalid
-//   transactions from the database when they are rejected by the network, other
-//   than double spending them.
+//   - The transaction is not inspected to be relevant before publishing using
+//     sendrawtransaction, so connection errors to btcd could result in the tx
+//     never being added to the wallet database.
+//   - Once the above bug is fixed, wallet will require a way to purge invalid
+//     transactions from the database when they are rejected by the network, other
+//     than double spending them.
 func (s *walletServer) PublishTransaction(ctx context.Context, req *pb.PublishTransactionRequest) (
 	*pb.PublishTransactionResponse, error) {
 

rpc/walletrpc/api.pb.go

@@ -6,7 +6,7 @@
 Package waddrmgr provides a secure hierarchical deterministic wallet address
 manager.
 
-Overview
+# Overview
 
 One of the fundamental jobs of a wallet is to manage addresses, private keys,
 and script data associated with them.  At a high level, this package provides
@@ -52,14 +52,14 @@ used to decrypt private keys and scripts on demand.  Relocking the address
 manager actively zeros all private material from memory.  In addition, temp
 private key material used internally is zeroed as soon as it's used.
 
-Locking and Unlocking
+# Locking and Unlocking
 
 As previously mentioned, this package provide facilities for locking and
 unlocking the address manager to protect access to private material and remove
 it from memory when locked.  The Lock, Unlock, and IsLocked functions are used
 for this purpose.
 
-Creating a New Address Manager
+# Creating a New Address Manager
 
 A new address manager is created via the Create function.  This function accepts
 a wallet database namespace, passphrases, network, and perhaps most importantly,
@@ -69,28 +69,28 @@ to be recovered with only the seed.  The GenerateSeed function in the hdkeychain
 package can be used as a convenient way to create a random seed for use with
 this function.  The address manager is locked immediately upon being created.
 
-Opening an Existing Address Manager
+# Opening an Existing Address Manager
 
 An existing address manager is opened via the Open function.  This function
 accepts an existing wallet database namespace, the public passphrase, and
 network.  The address manager is opened locked as expected since the open
 function does not take the private passphrase to unlock it.
 
-Closing the Address Manager
+# Closing the Address Manager
 
 The Close method should be called on the address manager when the caller is done
 with it.  While it is not required, it is recommended because it sanely shuts
 down the database and ensures all private and public key material is purged from
 memory.
 
-Managed Addresses
+# Managed Addresses
 
 Each address returned by the address manager satisifies the ManagedAddress
 interface as well as either the ManagedPubKeyAddress or ManagedScriptAddress
 interfaces.  These interfaces provide the means to obtain relevant information
 about the addresses such as their private keys and scripts.
 
-Chained Addresses
+# Chained Addresses
 
 Most callers will make use of the chained addresses for normal operations.
 Internal addresses are intended for internal wallet uses such as change outputs,
@@ -101,13 +101,13 @@ been provided.  In addition, the LastInternalAddress and LastExternalAddress
 functions can be used to get the most recently provided internal and external
 address, respectively.
 
-Requesting Existing Addresses
+# Requesting Existing Addresses
 
 In addition to generating new addresses, access to old addresses is often
 required.  Most notably, to sign transactions in order to redeem them.  The
 Address function provides this capability and returns a ManagedAddress.
 
-Importing Addresses
+# Importing Addresses
 
 While the recommended approach is to use the chained addresses discussed above
 because they can be deterministically regenerated to avoid losing funds as long
@@ -116,40 +116,40 @@ and as a result, this package provides the ability to import existing private
 keys in Wallet Import Format (WIF) and hence the associated public key and
 address.
 
-Importing Scripts
+# Importing Scripts
 
 In order to support pay-to-script-hash transactions, the script must be securely
 stored as it is needed to redeem the transaction.  This can be useful for a
 variety of scenarios, however the most common use is currently multi-signature
 transactions.
 
-Syncing
+# Syncing
 
 The address manager also supports storing and retrieving a block hash and height
 which the manager is known to have all addresses synced through.  The manager
 itself does not have any notion of which addresses are synced or not.  It only
 provides the storage as a convenience for the caller.
 
-Network
+# Network
 
 The address manager must be associated with a given network in order to provide
 appropriate addresses and reject imported addresses and scripts which don't
 apply to the associated network.
 
-Errors
+# Errors
 
 All errors returned from this package are of type ManagerError.  This allows the
 caller to programmatically ascertain the specific reasons for failure by
 examining the ErrorCode field of the type asserted ManagerError.  For certain
 error codes, as documented by the specific error codes, the underlying error
 will be contained in the Err field.
 
-Bitcoin Improvement Proposals
+# Bitcoin Improvement Proposals
 
 This package includes concepts outlined by the following BIPs:
 
-		BIP0032 (https://github.com/bitcoin/bips/blob/master/bip-0032.mediawiki)
-		BIP0043 (https://github.com/bitcoin/bips/blob/master/bip-0043.mediawiki)
-		BIP0044 (https://github.com/bitcoin/bips/blob/master/bip-0044.mediawiki)
+	BIP0032 (https://github.com/bitcoin/bips/blob/master/bip-0032.mediawiki)
+	BIP0043 (https://github.com/bitcoin/bips/blob/master/bip-0043.mediawiki)
+	BIP0044 (https://github.com/bitcoin/bips/blob/master/bip-0044.mediawiki)
 */
 package waddrmgr

waddrmgr/doc.go

@@ -7,6 +7,5 @@ Package wallet provides ...
 TODO: Flesh out this section
 
 Overview
-
 */
 package wallet

wallet/doc.go

@@ -9,15 +9,14 @@ import (
 	"encoding/hex"
 	"testing"
 
-	"github.com/btcsuite/btcwallet/wallet/txrules"
-	"github.com/btcsuite/btcwallet/wallet/txsizes"
-	"github.com/stretchr/testify/require"
-
 	"github.com/btcsuite/btcd/btcutil"
 	"github.com/btcsuite/btcd/btcutil/psbt"
 	"github.com/btcsuite/btcd/txscript"
 	"github.com/btcsuite/btcd/wire"
 	"github.com/btcsuite/btcwallet/waddrmgr"
+	"github.com/btcsuite/btcwallet/wallet/txrules"
+	"github.com/btcsuite/btcwallet/wallet/txsizes"
+	"github.com/stretchr/testify/require"
 )
 
 var (

wallet/psbt_test.go

@@ -189,11 +189,11 @@ func (rm *RecoveryManager) State() *RecoveryState {
 //
 // These are defined as:
 //   - Inter-Block Gap: The maximum difference between the derived child indexes
-//       of the last addresses used in any block and the next address consumed
-//       by a later block.
+//     of the last addresses used in any block and the next address consumed
+//     by a later block.
 //   - Intra-Block Gap: The maximum difference between the derived child indexes
-//       of the first address used in any block and the last address used in the
-//       same block.
+//     of the first address used in any block and the last address used in the
+//     same block.
 type RecoveryState struct {
 	// recoveryWindow defines the key-derivation lookahead used when
 	// attempting to recover the set of used addresses. This value will be
@@ -282,12 +282,12 @@ func NewScopeRecoveryState(recoveryWindow uint32) *ScopeRecoveryState {
 // derivation branch.
 //
 // A branch recovery state supports operations for:
-//  - Expanding the look-ahead horizon based on which indexes have been found.
-//  - Registering derived addresses with indexes within the horizon.
-//  - Reporting an invalid child index that falls into the horizon.
-//  - Reporting that an address has been found.
-//  - Retrieving all currently derived addresses for the branch.
-//  - Looking up a particular address by its child index.
+//   - Expanding the look-ahead horizon based on which indexes have been found.
+//   - Registering derived addresses with indexes within the horizon.
+//   - Reporting an invalid child index that falls into the horizon.
+//   - Reporting that an address has been found.
+//   - Retrieving all currently derived addresses for the branch.
+//   - Looking up a particular address by its child index.
 type BranchRecoveryState struct {
 	// recoveryWindow defines the key-derivation lookahead used when
 	// attempting to recover the set of addresses on this branch.

wallet/recovery.go

@@ -5,7 +5,7 @@
 /*
 Package walletdb provides a namespaced database interface for btcwallet.
 
-Overview
+# Overview
 
 A wallet essentially consists of a multitude of stored data such as private
 and public keys, key derivation bits, pay-to-script-hash scripts, and various
@@ -26,24 +26,24 @@ the wallet.
 
 A quick overview of the features walletdb provides are as follows:
 
- - Key/value store
- - Namespace support
- - Allows multiple packages to have their own area in the database without
-   worrying about conflicts
- - Read-only and read-write transactions with both manual and managed modes
- - Nested buckets
- - Supports registration of backend databases
- - Comprehensive test coverage
+  - Key/value store
+  - Namespace support
+  - Allows multiple packages to have their own area in the database without
+    worrying about conflicts
+  - Read-only and read-write transactions with both manual and managed modes
+  - Nested buckets
+  - Supports registration of backend databases
+  - Comprehensive test coverage
 
-Database
+# Database
 
 The main entry point is the DB interface.  It exposes functionality for
 creating, retrieving, and removing namespaces.  It is obtained via the Create
 and Open functions which take a database type string that identifies the
 specific database driver (backend) to use as well as arguments specific to the
 specified driver.
 
-Namespaces
+# Namespaces
 
 The Namespace interface is an abstraction that provides facilities for obtaining
 transactions (the Tx interface) that are the basis of all database reads and
@@ -55,14 +55,14 @@ The Begin function provides an unmanaged transaction while the View and Update
 functions provide a managed transaction.  These are described in more detail
 below.
 
-Transactions
+# Transactions
 
 The Tx interface provides facilities for rolling back or commiting changes that
 took place while the transaction was active.  It also provides the root bucket
 under which all keys, values, and nested buckets are stored.  A transaction
 can either be read-only or read-write and managed or unmanaged.
 
-Managed versus Unmanaged Transactions
+# Managed versus Unmanaged Transactions
 
 A managed transaction is one where the caller provides a function to execute
 within the context of the transaction and the commit or rollback is handled
@@ -75,7 +75,7 @@ call Commit or Rollback when they are finished with it.  Leaving transactions
 open for long periods of time can have several adverse effects, so it is
 recommended that managed transactions are used instead.
 
-Buckets
+# Buckets
 
 The Bucket interface provides the ability to manipulate key/value pairs and
 nested buckets as well as iterate through them.
@@ -85,15 +85,15 @@ CreateBucket, CreateBucketIfNotExists, and DeleteBucket functions work with
 buckets.  The ForEach function allows the caller to provide a function to be
 called with each key/value pair and nested bucket in the current bucket.
 
-Root Bucket
+# Root Bucket
 
 As discussed above, all of the functions which are used to manipulate key/value
 pairs and nested buckets exist on the Bucket interface.  The root bucket is the
 upper-most bucket in a namespace under which data is stored and is created at
 the same time as the namespace.  Use the RootBucket function on the Tx interface
 to retrieve it.
 
-Nested Buckets
+# Nested Buckets
 
 The CreateBucket and CreateBucketIfNotExists functions on the Bucket interface
 provide the ability to create an arbitrary number of nested buckets.  It is