Make all keepers pointers, use multiple initialization methods (#697)

* Make all keepers pointers, use multiple initialization methods * Split out new store loader logic * Make antehandler input simpler * Improve comment * Improve comment
osmosis-labs · Jan 9, 2022 · 343bce9 · 343bce9
1 parent 5a5cc38
commit 343bce9
Show file tree

Hide file tree

Showing 24 changed files with 490 additions and 511 deletions.
diff --git a/app/ante.go b/app/ante.go
@@ -19,7 +19,7 @@ import (
 // https://github.com/cosmos/cosmos-sdk/blob/v0.43.0/x/auth/ante/ante.go#L41
 func NewAnteHandler(
 	ak ante.AccountKeeper, bankKeeper authtypes.BankKeeper,
-	txFeesKeeper txfeeskeeper.Keeper, spotPriceCalculator txfeestypes.SpotPriceCalculator,
+	txFeesKeeper *txfeeskeeper.Keeper, spotPriceCalculator txfeestypes.SpotPriceCalculator,
 	sigGasConsumer ante.SignatureVerificationGasConsumer,
 	signModeHandler signing.SignModeHandler,
 	channelKeeper channelkeeper.Keeper,
@@ -30,7 +30,7 @@ func NewAnteHandler(
 		NewMempoolMaxGasPerTxDecorator(),
 		// Use Mempool Fee Decorator from our txfees module instead of default one from auth
 		// https://github.com/cosmos/cosmos-sdk/blob/master/x/auth/middleware/fee.go#L34
-		txfeeskeeper.NewMempoolFeeDecorator(txFeesKeeper),
+		txfeeskeeper.NewMempoolFeeDecorator(*txFeesKeeper),
 		ante.NewValidateBasicDecorator(),
 		ante.TxTimeoutHeightDecorator{},
 		ante.NewValidateMemoDecorator(ak),