Fixup comments
This commit is contained in:
parent
317be2bc16
commit
1ed338fc5f
|
@ -58,7 +58,7 @@ func (k Keeper) handleDoubleSign(ctx sdk.Context, addr crypto.Address, infractio
|
||||||
logger.Info(fmt.Sprintf("Confirmed double sign from %s at height %d, age of %d less than max age of %d", pubkey.Address(), infractionHeight, age, maxEvidenceAge))
|
logger.Info(fmt.Sprintf("Confirmed double sign from %s at height %d, age of %d less than max age of %d", pubkey.Address(), infractionHeight, age, maxEvidenceAge))
|
||||||
|
|
||||||
// We need to retrieve the stake distribution which signed the block, so we subtract ValidatorUpdateDelay from the evidence height.
|
// We need to retrieve the stake distribution which signed the block, so we subtract ValidatorUpdateDelay from the evidence height.
|
||||||
// Note that this *can* result in a "distributionHeight" of -1,
|
// Note that this *can* result in a negative "distributionHeight", up to -ValidatorUpdateDelay,
|
||||||
// i.e. at the end of the pre-genesis block (none) = at the beginning of the genesis block.
|
// i.e. at the end of the pre-genesis block (none) = at the beginning of the genesis block.
|
||||||
// That's fine since this is just used to filter unbonding delegations & redelegations.
|
// That's fine since this is just used to filter unbonding delegations & redelegations.
|
||||||
distributionHeight := infractionHeight - stake.ValidatorUpdateDelay
|
distributionHeight := infractionHeight - stake.ValidatorUpdateDelay
|
||||||
|
@ -135,7 +135,7 @@ func (k Keeper) handleValidatorSignature(ctx sdk.Context, addr crypto.Address, p
|
||||||
pubkey.Address(), minHeight, k.MinSignedPerWindow(ctx)))
|
pubkey.Address(), minHeight, k.MinSignedPerWindow(ctx)))
|
||||||
// We need to retrieve the stake distribution which signed the block, so we subtract ValidatorUpdateDelay from the evidence height,
|
// We need to retrieve the stake distribution which signed the block, so we subtract ValidatorUpdateDelay from the evidence height,
|
||||||
// and subtract an additional 1 since this is the LastCommit.
|
// and subtract an additional 1 since this is the LastCommit.
|
||||||
// Note that this *can* result in a "distributionHeight" of -1 or -2,
|
// Note that this *can* result in a negative "distributionHeight" up to -ValidatorUpdateDelay-1,
|
||||||
// i.e. at the end of the pre-genesis block (none) = at the beginning of the genesis block.
|
// i.e. at the end of the pre-genesis block (none) = at the beginning of the genesis block.
|
||||||
// That's fine since this is just used to filter unbonding delegations & redelegations.
|
// That's fine since this is just used to filter unbonding delegations & redelegations.
|
||||||
distributionHeight := height - stake.ValidatorUpdateDelay - 1
|
distributionHeight := height - stake.ValidatorUpdateDelay - 1
|
||||||
|
|
Loading…
Reference in New Issue