[proxy] web.archive.org — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link
Contributor

asvetlov commented Dec 26, 2018 •
edited by bedevere-bot
Loading

asvetlov requested a review from ethanfurman December 26, 2018 00:32
ethanfurman self-assigned this Dec 26, 2018
ethanfurman changed the title bpo-35585: Speed-up building enums by value, e.g. http.HTTPStatus(200) bpo-35585: Speed-up enum by-value lookup Dec 26, 2018
ethanfurman changed the title bpo-35585: Speed-up enum by-value lookup bpo-35585: Speed up enum by-value lookup Dec 26, 2018
ethanfurman merged commit 34ae04f into python:master Dec 26, 2018
Copy link

bedevere-bot commented Dec 26, 2018

@ethanfurman: Please replace # with GH- in the commit message next time. Thanks!

Copy link
Contributor

miss-islington commented Dec 26, 2018

Thanks @asvetlov for the PR, and @ethanfurman for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

Copy link

bedevere-bot commented Dec 26, 2018

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

ethanfurman added a commit that referenced this pull request Dec 26, 2018
Revert "Speed-up building enums by value, e.g. http.HTTPStatus(200) (#…
ba6ea52 
…11318)"

This reverts commit 34ae04f.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 26, 2018
Speed-up building enums by value, e.g. http.HTTPStatus(200) (pythonGH…
5447a2e 
…-11318)

bpo-35585: Speed up enum by-value lookup
(cherry picked from commit 34ae04f)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
asvetlov deleted the fast-enum branch December 26, 2018 20:45
asvetlov added a commit that referenced this pull request Dec 26, 2018
Speed-up building enums by value, e.g. http.HTTPStatus(200) (GH-11318) (
705b599 
GH-11324)

bpo-35585: Speed up enum by-value lookup
(cherry picked from commit 34ae04f)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
tirkarthi mentioned this pull request Jan 8, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

tirkarthi tirkarthi approved these changes

ethanfurman ethanfurman approved these changes

Assignees

ethanfurman

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants