Skip to content

Commit

Permalink
Remove unused argument.
Browse files Browse the repository at this point in the history
  • Loading branch information
DimitrisJim committed Jul 25, 2023
1 parent d6ac7fc commit db3eefb
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions modules/light-clients/08-wasm/types/client_state.go
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ func (cs ClientState) VerifyMembership(
return errorsmod.Wrapf(ibcerrors.ErrInvalidType, "expected %T, got %T", commitmenttypes.MerklePath{}, path)
}

found := hasConsensusState(clientStore, cdc, height)
found := hasConsensusState(clientStore, height)
if !found {
return errorsmod.Wrapf(clienttypes.ErrConsensusStateNotFound, "please ensure the proof was constructed against a height that exists on the client")
}
Expand Down Expand Up @@ -233,7 +233,7 @@ func (cs ClientState) VerifyNonMembership(
return errorsmod.Wrapf(ibcerrors.ErrInvalidType, "expected %T, got %T", commitmenttypes.MerklePath{}, path)
}

found := hasConsensusState(clientStore, cdc, height)
found := hasConsensusState(clientStore, height)
if !found {
return errorsmod.Wrapf(clienttypes.ErrConsensusStateNotFound, "please ensure the proof was constructed against a height that exists on the client")
}
Expand Down
2 changes: 1 addition & 1 deletion modules/light-clients/08-wasm/types/store.go
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ func setConsensusState(clientStore sdk.KVStore, cdc codec.BinaryCodec, consensus
}

// hasConsensusState returns true if a consensus state at the given height exists in the store.
func hasConsensusState(clientStore sdk.KVStore, cdc codec.BinaryCodec, height exported.Height) bool {
func hasConsensusState(clientStore sdk.KVStore, height exported.Height) bool {
return clientStore.Has(host.ConsensusStateKey(height))
}

Expand Down
2 changes: 1 addition & 1 deletion modules/light-clients/08-wasm/types/upgrade.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ func (cs ClientState) VerifyUpgradeAndUpdateState(
// Must prove against latest consensus state to ensure we are verifying against latest upgrade plan
// This verifies that upgrade is intended for the provided revision, since committed client must exist
// at this consensus state
found := hasConsensusState(clientStore, cdc, lastHeight)
found := hasConsensusState(clientStore, lastHeight)
if !found {
return errorsmod.Wrapf(clienttypes.ErrConsensusStateNotFound, "could not retrieve consensus state for height %s", lastHeight)
}
Expand Down

0 comments on commit db3eefb

Please sign in to comment.