Skip to content

Conversation

bbayles
Copy link
Contributor

@bbayles bbayles commented Jan 24, 2018

This PR backports #5254 to Python 3.6, per this discussion. I think I got the cherry_picker script right.

https://bugs.python.org/issue32502

Copy link
Member

@warsaw warsaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Let's see how CI finishes.

if _node is not None:
if (_node is not None) and (0 <= _node < (1 << 48)):
return _node
assert False, '_random_getnode() returned invalid value: {}'.format(_node)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use f-expression ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe; I think in this case it won't matter much either way since the whole point is to never reach this assert. :)

@warsaw warsaw merged commit d69794f into python:3.6 Jan 24, 2018
@bbayles bbayles deleted the backport-6b273f7-3.6 branch January 26, 2018 00:13
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.

5 participants