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
cmath.polar() can raise due to pre-existing errno #68677
Comments
Here is a patch, with additional tests. |
polar_errno.patch doesn't apply cleanly on the default branch, I get that you wrote your patch for Python 3.4. + @cpython_only Please add the number of this issue in the comment. + inf = float('inf') On Python 3.5 and 3.6, You can use math.inf and math.nan. Except of that, the patch looks good to me. |
The patch is for 3.4. |
Ok, it confirms my guess. But when you merge your change into 3.5, you may replace nan and inf with math.nan and math.inf. As you want. |
New changeset 70e3230c2872 by Antoine Pitrou in branch '3.4': New changeset d165712b2dee by Antoine Pitrou in branch '3.5': New changeset b1fac7685947 by Antoine Pitrou in branch 'default': |
New changeset 9f4b066754c3 by Antoine Pitrou in branch '2.7': |
Ok, pushed it! |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: