Question about eligible validators in rewards/penalties processing

[NIC619]

In spec v0.5.1, eligible validators are active validators in previous epoch:

    # v0.5.1
    # Process rewards or penalties for all validators
    for index in get_active_validator_indices(state.validator_registry, get_previous_epoch(state)):
        ...

However they are changed to current epoch active validators in d1d1b73 but not mentioned in the commit message or PR description so I wonder if this is intended?

    # dev
    eligible_validators = [
        index for index, validator in enumerate(state.validator_registry)
        if (
            is_active_validator(validator, current_epoch) or
            (validator.slashed and current_epoch < validator.withdrawable_epoch)
        )
    ]
    # Process rewards or penalties for all validators
    for index in eligible_validators:

And does it makes sense to change all validators in the comment # Process rewards or penalties for all validators to be something more specific?

[djrtwo]

You're right, this would be the previous epoch

[djrtwo]

This is being addressed in #949

[hwwhww]

closing in favor of #949