bpo-40434: Update the reason why CPython doesn't have switch statements#19787
bpo-40434: Update the reason why CPython doesn't have switch statements#19787amaajemyfren wants to merge 2 commits intopython:masterfrom amaajemyfren:design_faq
Conversation
|
Thanks @remilapeyre for the approval. Kindly add the skip news label and advise on the next steps please. Thank you. |
The skip news label will be added by a core developer that will make a second review of your PR and either ask for some changes or merge it :) |
There was a problem hiding this comment.
@amaajemyfren , I commented on bpo but decided to put the same thought here. I understand the change but curious what is your reasoning on referencing just one of the PEPs and not both of them?
|
Hi @nanjekyejoanna, the statement initially said that there was no consensus yet on how to do range tests. This is not true because there is now a decision to not do range tests - that decision is only in PEP 3103 and not in PEP 275 (PEP 275 actually links to PEP 3103 to explain why it is rejected). My feeling is that putting two references complicates the goal of explaining it. That said there is also this but that may still be a while away. |
nanjekyejoannah
left a comment
nanjekyejoannah
left a comment
There was a problem hiding this comment.
PEP 275 actually links to PEP 3103 to explain why it is rejected
Well, am not very convinced if consensus was reached on range tests so I will refrain for someone else's opinion as it is not as apparent to me yet.
|
@iritkatriel Was this an accident? Seems like the section has been substantially updated, with these sentences eliminated and replaced with information about the new |
|
Thanks for checking and closing. |
|
(N.B., Victor closed it 2 years ago, not me :) |
|
Ah, I thought this was the issue - I guess we can close that now: #84614 |
|
Ah, gotcha—didn't see the linked issue. Thanks! |
8 participants
https://bugs.python.org/issue40434