-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
Closed
Labels
3.12only security fixesonly security fixesbuildThe build process and cross-buildThe build process and cross-build
Description
Issue #100222 added a struct to Include/cpython/code.h
(which is included by default by Python.h
) that is not compliant C++:
typedef union {
uint16_t cache;
struct {
uint8_t opcode;
uint8_t oparg;
};
} _Py_CODEUNIT;
The struct member requires a name, as only anonymous structs are currently permitted in standard C++.
(And since the original issue invokes PEP 7, I'll do the same and point out that "The public C API should be compatible with C++". The intention of this clause is that standard C++ can #include <Python.h>
, which is currently broken.)
Linked PRs
Metadata
Metadata
Assignees
Labels
3.12only security fixesonly security fixesbuildThe build process and cross-buildThe build process and cross-build