Skip to content

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commented Jul 23, 2023

Rename Include/errcode.h to Include/internal/pycore_errcode.h.


📚 Documentation preview 📚: https://cpython-previews--107163.org.readthedocs.build/

@vstinner
Copy link
MemberAuthor

cc @pablogsal@lysnikolaou@serhiy-storchaka: see #107162 for the removal rationale.

Rename Include/errcode.h to Include/internal/pycore_errcode.h.
serhiy-storchaka
serhiy-storchaka previously approved these changes Jul 24, 2023
Copy link
Member

@serhiy-storchakaserhiy-storchaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is a part of the C API. These codes are returned by PyRun_InteractiveOneFlags(). See Doc/c-api/veryhigh.rst.

@vstinner
Copy link
MemberAuthor

So this PR is wrong. I wrote PR #107177 to complete the comment instead.

Thanks @serhiy-storchaka for your review!

@vstinnervstinner deleted the remove_errcode_h branch July 24, 2023 13:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@serhiy-storchaka@bedevere-bot