cerc-io/lighthouse
16
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

attester_duties: remove unnecessary case (#4614)

Browse Source 
Since `tolerant_current_epoch` is expected to be either `current_epoch` or `current_epoch+1`, we can eliminate a case here. 

And added a comment about `compute_historic_attester_duties` , since `RelativeEpoch::from_epoch` will only allow `request_epoch == current_epoch-1` when `request_epoch < current_epoch`.
This commit is contained in:
zhiqiangxu 2023-08-17 02:37:30 +00:00
parent 7251a93c5e
commit 609819bb4d
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
beacon_node/http_api/src/attester_duties.rs
View File

@ -35,7 +35,6 @@ pub fn attester_duties<T: BeaconChainTypes>(
.epoch(T::EthSpec::slots_per_epoch()); .epoch(T::EthSpec::slots_per_epoch());
if request_epoch == current_epoch if request_epoch == current_epoch
|| request_epoch == tolerant_current_epoch
|| request_epoch == current_epoch + 1 || request_epoch == current_epoch + 1
|| request_epoch == tolerant_current_epoch + 1 || request_epoch == tolerant_current_epoch + 1
{ {
@ -46,7 +45,7 @@ pub fn attester_duties<T: BeaconChainTypes>(
request_epoch, current_epoch request_epoch, current_epoch
))) )))
} else { } else {
// request_epoch < current_epoch // request_epoch < current_epoch, in fact we only allow `request_epoch == current_epoch-1` in this case
compute_historic_attester_duties(request_epoch, request_indices, chain) compute_historic_attester_duties(request_epoch, request_indices, chain)
} }
} }