-
Notifications
You must be signed in to change notification settings - Fork 766
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove arbitrary precision feature #2617
Conversation
fixing clippy rn |
I know now why this was added... geth emits large ints for |
@Rjected should we remove the totaldifficulty from the object? |
We probably can, IIRC this is not required for spawning geth with a normal genesis, and we don't use ethers for hive tests / hive genesis |
337b7eb
to
747bbe8
Compare
@Rjected turns out this can be fixed by manually converting the captured float, let's do that on reth as well |
3328fdd
to
c9d68be
Compare
c9d68be
to
7bac093
Compare
closes #2206
this feature is kinda horrible and we should not force anyone into using it.
this is also only useful when deserializing json large numbers but since u256 are supposed to be hex it's fine to not support deserializing arbitrary numbers