-
-
Notifications
You must be signed in to change notification settings - Fork 844
Fix links to missing CPython files #1003
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Not removed, read the commit: it was moved to Include/internal/pycore_ast.h.
Same, just renamed to to Include/internal/pycore_token.h. I prefer to say "removed" for end users, since end users are not supposed to consume the internal C API.
This file was basically empty: you should use Include/cpython/code.h instead. |
Does this issue require more changes? |
I believe so, as most of the existing PRs focused on the existing |
Got it. So for example here
Am I looking at the right place? |
Those are examples of things you would normally want to submit a PR changing, yes, but also examples of things on the C-API page that I mentioned is currently being modified by #1060 and will be further modified by a followup, so you shouldn't touch those until that's all done. If you find similar such references in other documents, those would likely be fair game to change as part of one or more PRs. |
Alright, got it. Thank you. |
This is a follow-up of #984:
I found another file mentioned in the devguide that was deleted:
Python/wordcode_helpers.h
was removed by @brandtbucherBINARY_OP
cpython#31543The text was updated successfully, but these errors were encountered: