Skip to content

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

Merged
JelleZijlstra merged 1 commit into
python:3.12from
miss-islington:backport-796b3fb-3.12
Apr 26, 2024
Merged

[3.12] gh-118207: Rename the COMMON_FIELDS macro in funcobject.h and undef it after use (GH-118208)#118269
JelleZijlstra merged 1 commit into
python:3.12from
miss-islington:backport-796b3fb-3.12

Conversation

@miss-islington
Copy link
Copy Markdown
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
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
@miss-islington miss-islington deleted the backport-796b3fb-3.12 branch January 2, 2026 17:48
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