next_validator_index comment is misleading
Closed this issue · 2 comments
paulhauner commented
This var does not represent the number of validators as it is set to 1 in the constructor. It technically represents number of validators + 1.
I would suggest changing the comment to "Next available validator index.", however I am no authority 😄
ChihChengLiang commented
the comment "Next available validator index." sounds good!
djrtwo commented
It doesn't even represent the number of validators + 1! It represents the number of validators there have ever been (+ 1)...
changing it now