Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Jun 6, 2018

Move the floor() call into fbound() to call floor() on a double
rather than an int. The change might enhance the rounding.

https://bugs.python.org/issue33781

Move the floor() call into fbound() to call floor() on a double
rather than an int. The change might enhance the rounding.
@vstinner
Copy link
Member Author

vstinner commented Jun 6, 2018

VSTS: Windows-PR failed with an internal error: https://bugs.python.org/issue33782

@vstinner vstinner merged commit 45e4efb into python:master Jun 6, 2018
@vstinner vstinner deleted the audioop_int branch June 6, 2018 13:50
@serhiy-storchaka serhiy-storchaka added type-bug An unexpected behavior, bug, or error needs backport to 3.6 labels Jun 6, 2018
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 2.7.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot
Copy link

GH-7450 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 6, 2018
Move the floor() call into fbound() to call floor() on a double
rather than an int. The change should enhance the rounding.

Document also (int)double rounding mode.
(cherry picked from commit 45e4efb)

Co-authored-by: Victor Stinner <[email protected]>
@bedevere-bot
Copy link

GH-7451 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 6, 2018
Move the floor() call into fbound() to call floor() on a double
rather than an int. The change should enhance the rounding.

Document also (int)double rounding mode.
(cherry picked from commit 45e4efb)

Co-authored-by: Victor Stinner <[email protected]>
@miss-islington
Copy link
Contributor

Sorry, @vstinner, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 45e4efba7fa2abe61d25e4f8b5bf482e19ff1280 2.7

miss-islington added a commit that referenced this pull request Jun 6, 2018
Move the floor() call into fbound() to call floor() on a double
rather than an int. The change should enhance the rounding.

Document also (int)double rounding mode.
(cherry picked from commit 45e4efb)

Co-authored-by: Victor Stinner <[email protected]>
@bedevere-bot
Copy link

GH-7452 is a backport of this pull request to the 2.7 branch.

vstinner added a commit that referenced this pull request Jun 6, 2018
Move the floor() call into fbound() to call floor() on a double
rather than an int. The change should enhance the rounding.

Document also (int)double rounding mode.

(cherry picked from commit 45e4efb)
vstinner added a commit that referenced this pull request Jun 6, 2018
Move the floor() call into fbound() to call floor() on a double
rather than an int. The change should enhance the rounding.

Document also (int)double rounding mode.
(cherry picked from commit 45e4efb)

Co-authored-by: Victor Stinner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants