Skip to content

[3.12] gh-118207: Rename the COMMON_FIELDS macro in funcobject.h and undef it after use (GH-118208) #118269

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
Apr 26, 2024

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Apr 25, 2024

(cherry picked from commit 796b3fb)

Co-authored-by: Itamar Oren itamarost@gmail.com

…ndef it after use (pythonGH-118208)

(cherry picked from commit 796b3fb)

Co-authored-by: Itamar Oren <itamarost@gmail.com>
@JelleZijlstra JelleZijlstra merged commit 82e26d6 into python:3.12 Apr 26, 2024
27 checks passed
facebook-github-bot pushed a commit to facebookincubator/cinder that referenced this pull request Apr 26, 2024
…nd undef it after use

Summary:
supersedes D56454529 by backporting the upstream version instead of an internal patch

upstream issue: python/cpython#118207
upstream PR: python/cpython#118269

still undef'ing the macro, but also renaming it to have `_Py` prefix, for consistency, and to avoid conflicts if the same name is already defined before including `Python.h`

Reviewed By: aleivag

Differential Revision: D56579827

fbshipit-source-id: c8260818f5901bc9c3eb582999575af9c64f189f
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.

3 participants