Uh oh!
There was an error while loading. Please reload this page.
gh-114258: Refactor Argument Clinic function name parser - #114930
Conversation
Yak-shaving in preparation for adding support for slots. - factor out return converter resolver - factor out cloning
erlend-aasland
commented
Feb 2, 2024
(I'll fix the mypy errors later) |
This comment was marked as resolved.
This comment was marked as resolved.
erlend-aasland
commented
Feb 3, 2024
FTR, most of this PR is just shuffling lines around and wrapping them in more targeted scopes. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
erlend-aasland
commented
Feb 5, 2024
Did you reject #114930 (comment) or missed it? :) Update: Ok, I see that it was partially implemented :) |
sobolevn
left a comment
There was a problem hiding this comment.
Thanks! We can work on better error messages in the next PRs.
erlend-aasland
commented
Feb 5, 2024
Yeah, I think we should try to do an audit of all the |
erlend-aasland
commented
Feb 5, 2024
Thanks for the review, Nikita. I'll wait a day or two to give Alex a chance to chime in (since I already pinged him about this). |
erlend-aasland
commented
Feb 7, 2024
Landing this tomorrow afternoon (CET). |
Refactor state_modulename_name() of the parsing state machine, by
adding helpers for the sections that deal with ...: