ethereum/py_ecc
Cleanup isinstance checks for integers to disallow booleans
Aberta
#30 aberto em 4 de dez. de 2018
Good First Issue
Métricas do repositório
- Stars
- (231 estrelas)
- Métricas de merge de PR
- (Métricas PR pendentes)
Description
What is wrong?
Lots of places in the codebase do assertions like isinstance(v, int) to check that something is an integer. Since python considers True and False to be integer types these checks won't disallow passing in a bool value.
How can it be fixed
Probably need to add a single utility and make use of it everywhere that we do these checks.
def assert_strictly_integer(v):
assert isinstance(v, int) and not isinstance(v, bool)