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.

Sign up for GitHub

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

Jump to bottom

Improve error messages and argument checking in .imuln() #207

Open

webmaster128 wants to merge 3 commits into indutny:master from webmaster128:muln-asserts

webmaster128 commented Nov 29, 2018

I got an "Assertion failed" after doing a .muln(10**9), which was not very helpful. Thus this improved error messages and some more checks.

Simon Warta added 3 commits

November 29, 2018 15:22


          Improve error messages for imuln argument range

c62a93b


          Improve error message and add test for imuln argument type

1d299e5


          Ensure argument of imuln is an integer

c8766d6

BertsLifee approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet