I think we should have a consistent approach of validating signer and timestamp.
I prefer the isValid... approach that return true or false, as we could pre-validate the arguments and test methods from outside.
I see, the value in returning "data in the future" - but this is an anomaly and it's fine to revert if something that should never happen.
I think we should have a consistent approach of validating signer and timestamp. I prefer the isValid... approach that return true or false, as we could pre-validate the arguments and test methods from outside. I see, the value in returning "data in the future" - but this is an anomaly and it's fine to revert if something that should never happen.