Skip to content

Conversation

@cdetrio
Copy link

@cdetrio cdetrio commented Sep 7, 2017

No description provided.

@cdetrio
Copy link
Author

cdetrio commented Sep 8, 2017

to_big_endian was returning numbers in Montgomery representation. This converts them to standard, as ecadd and ecmul precompiles expect ethereum/EIPs#213

@arkpar
Copy link
Member

arkpar commented Sep 8, 2017

Thank you, I've confirmed that the tests are passing now. @NikVolf This should be an additional conversion function though as ethereum precompile representation is of no concern to this library.

@NikVolf NikVolf merged commit f121cb5 into paritytech:master Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants