Skip to content

gh-108765: Move stat() fiddling from pyport.h to fileutils.h #108854

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

Merged
merged 1 commit into from
Sep 3, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Sep 3, 2023

@vstinner
Copy link
Member Author

vstinner commented Sep 3, 2023

This change is safe. It does not change any API, it only moves code from pyport.h to fileutils.h, but both header files are included by Python.h (which is the only header which must be included).

@vstinner vstinner enabled auto-merge (squash) September 3, 2023 21:14
@vstinner vstinner merged commit c2ec174 into python:main Sep 3, 2023
@vstinner vstinner deleted the stat_fiddling branch September 3, 2023 21:32
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 27, 2025
…pythonGH-108854 (pythonGH-136027)

(cherry picked from commit 0141e7f)

Co-authored-by: Lee Dogeon <dev.moreal@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 27, 2025
…pythonGH-108854 (pythonGH-136027)

(cherry picked from commit 0141e7f)

Co-authored-by: Lee Dogeon <dev.moreal@gmail.com>
picnixz pushed a commit that referenced this pull request Jun 27, 2025
GH-108854 (GH-136027) (#136044)

gh-108765: fix comment about macro definitions in `_stat.c` post GH-108854 (GH-136027)
(cherry picked from commit 0141e7f)

Co-authored-by: Lee Dogeon <dev.moreal@gmail.com>
picnixz pushed a commit that referenced this pull request Jun 27, 2025
GH-108854 (GH-136027) (#136043)

gh-108765: fix comment about macro definitions in `_stat.c` post GH-108854 (GH-136027)
(cherry picked from commit 0141e7f)

Co-authored-by: Lee Dogeon <dev.moreal@gmail.com>
Sign up for free to 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.

2 participants