Uh oh!
There was an error while loading. Please reload this page.
gh-113317: Add ParseArgsCodeGen class - #117707
Conversation
vstinner
commented
Apr 10, 2024
This PR is based on PR gh-117626. |
781133c to
b17f552Comparevstinner
commented
Apr 11, 2024
PR rebased on top of merged #117626. |
erlend-aasland
commented
Apr 11, 2024
The previous PR used the |
erlend-aasland
commented
Apr 11, 2024
This is nice! Should we consider adding this as a separate module? Or would |
vstinner
commented
Apr 11, 2024
CLanguage uses ParseArgsCodeGen which uses CLanguage... There is an inter-dependency. It's more convenient to have both classes in the same file. |
erlend-aasland
commented
Apr 11, 2024
Yes, there are inter-dependencies everywhere :( |
vstinner
commented
Apr 11, 2024
I updated this PR to rename Codegen to CodeGen. |
vstinner
commented
Apr 11, 2024
I broke the inter-dependency between CLanguage and ParseArgsCodeGen, and I moved declare_parser() and ParseArgsCodeGen to a new |
erlend-aasland
commented
Apr 11, 2024
Nice, so now we can make the parser templates globals of the |
vstinner
commented
Apr 11, 2024
Done. |
Uh oh!
There was an error while loading. Please reload this page.