Message372671
> From what I understand, "checking for two or more hanging INDENTS" and, "hardcoding a check for nonlocal SyntaxErrors in codeop._maybe_compile" are two different solutions, right? If yes, do we have an answer to which one of them is more cleaner, and henceforth, the preferable solution?
Note that these are two solution that take very different approaches. What Nick is suggesting with "checking for two or more hanging INDENTS" would drastically change how codeop._maybe_compile does its thing, while his other proposed solution, ""hardcoding a check for nonlocal SyntaxErrors in codeop._maybe_compile", would just fix this issue.
> I, personally, like the idea of checking INDENTS primarily because of it's reduced specificity, but I am in no position to comment on this (I already kinda did ':D), and you folks know better! For all we know, we should be optimizing for specificity.
You're right that this idea is more general. It would require significantly more effort from whoever tackles this though.
> Also, reading Nick's comments and the comc's code, gives me the feeling that a fix for this wouldn't require drastic changes.
That really depends on what solution is chosen out of the two.
> I'm slowly starting my journey with CPython, and I'd like to contribute a patch if that is the case. Thanks!
If there's consensus around one proposed approach, you could certainly take this up. I'd be glad to help out with workflow stuff or provide a first review. Let me know if you've got more questions. |
|
Date |
User |
Action |
Args |
2020-06-30 09:13:56 | lys.nikolaou | set | recipients:
+ lys.nikolaou, terry.reedy, ncoghlan, r.david.murray, Mark.Shannon, Rosuav, Esa.Peuha, mbussonn, cheryl.sabella, pablogsal, RJ722 |
2020-06-30 09:13:56 | lys.nikolaou | set | messageid: <1593508436.15.0.345277798622.issue19335@roundup.psfhosted.org> |
2020-06-30 09:13:56 | lys.nikolaou | link | issue19335 messages |
2020-06-30 09:13:55 | lys.nikolaou | create | |
|