8000 3-argument pow fails with newint arguments in Python 2 · Issue #87 · PythonCharmers/python-future · GitHub
[go: up one dir, main page]
Skip to content

3-argument pow fails with newint arguments in Python 2 #87

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
Closed
str4d opened this issue Aug 8, 2014 · 1 comment
Closed

3-argument pow fails with newint arguments in Python 2 #87

str4d opened this issue Aug 8, 2014 · 1 comment

Comments

@str4d
Copy link
Contributor
str4d commented Aug 8, 2014

In Python 3, future.builtins.int == int, and pow operates as expected.
In Python 2, future.builtins.int == newint and the 3-argument version of pow fails.

The 2-argument version works fine; it seems that in Python 2, the 2-argument version will take any int-like argument, but the 3-argument version will only take int:

>>> pow(2, 3)
8
>>> pow(int(2), 3)
8
>>> pow(int(2), 3, 3)
TypeError: __pow__() takes exactly 2 arguments (3 given)
>>> pow(2, int(3), 3)
TypeError: unsupported operand type(s) for pow(): 'int', 'newint', 'int'

Looking at the source, future.builtins.pow uses the Python 2 pow transparently, when it should be handling the presence of newints.
str4d added a commit to str4d/python-future that referenced this issue Aug 8, 2014
@str4d
Wrap Python 2 pow(x, y, z) args in long() to protect from newints (is…
970ae0c 
…sue PythonCharmers#87)
@str4d str4d mentioned this issue Aug 8, 2014
Closed
edschofield added a commit that referenced this issue Aug 13, 2014
@edschofield
Update What's New doc to mention str4d's fix for issue #87 and add st…
d80cea9 
…r4d to credits page
@edschofield
Copy link
Contributor

Fixed in v0.13.

@xmfbit xmfbit mentioned this issue Feb 3, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@edschofield @str4d
0