From b80d6e049599355f3ac214d0aba5a645baa4a570 Mon Sep 17 00:00:00 2001 From: Alex Stokes Date: Wed, 10 Jul 2019 17:00:11 -0700 Subject: [PATCH] Avoid unnecessary materialization of list There is a realization of a `list` in the `get_unslashed_attesting_indices` helper that is unnecessary. The functionality in this PR is the same so this change should only really be cosmetic wrt the spec. --- specs/core/0_beacon-chain.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/specs/core/0_beacon-chain.md b/specs/core/0_beacon-chain.md index c0d52463a..e2d734e13 100644 --- a/specs/core/0_beacon-chain.md +++ b/specs/core/0_beacon-chain.md @@ -1271,7 +1271,7 @@ def get_unslashed_attesting_indices(state: BeaconState, output = set() # type: Set[ValidatorIndex] for a in attestations: output = output.union(get_attesting_indices(state, a.data, a.aggregation_bits)) - return set(filter(lambda index: not state.validators[index].slashed, list(output))) + return set(filter(lambda index: not state.validators[index].slashed, output)) ``` ```python