avoid pointless rangeerror checks each iteration

status-im · Sep 10, 2023 · 9ba6558 · 9ba6558
1 parent 5ce9ae4
commit 9ba6558
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/beacon_chain/spec/datatypes/base.nim b/beacon_chain/spec/datatypes/base.nim
@@ -669,12 +669,14 @@ template `[]`*[T](a: seq[T], b: ValidatorIndex): auto = # Also var seq (!)
 
 iterator vindices*(
     a: HashList[Validator, Limit VALIDATOR_REGISTRY_LIMIT]): ValidatorIndex =
-  for i in 0..<a.len():
+  static: doAssert distinctBase(ValidatorIndex) is uint32
+  for i in 0..<a.len.uint32:
     yield i.ValidatorIndex
 
 iterator vindices*(
     a: List[Validator, Limit VALIDATOR_REGISTRY_LIMIT]): ValidatorIndex =
-  for i in 0..<a.len():
+  static: doAssert distinctBase(ValidatorIndex) is uint32
+  for i in 0..<a.len.uint32:
     yield i.ValidatorIndex
 
 template `==`*(x, y: JustificationBits): bool =