8000 MNT: always require va_start to have two arguments (#4820) · cython/cython@ad2d1f7 · GitHub
[go: up one dir, main page]

Skip to content

Commit ad2d1f7

Browse files
tacaswellda-woods
authored andcommitted
MNT: always require va_start to have two arguments (#4820)
* MNT: always require va_start to have two arguments python/cpython#93215 chance CPython to always use the 2-input version of va_start and dropped defining HAVE_STDARG_PROTOTYPES. This resulted in the 1-argument version being used when compiling cython source which fails This makes cython also always use the 2-argument version. * Remove blank line * FIX: version gate 2-argument va_start checking to py311
1 parent c0dfb9c commit ad2d1f7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Cython/Utility/MemoryView_C.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -450,7 +450,7 @@ static void __pyx_fatalerror(const char *fmt, ...) Py_NO_RETURN {
450450
va_list vargs;
451451
char msg[200];
452452

453-
#ifdef HAVE_STDARG_PROTOTYPES
453+
#if PY_VERSION_HEX >= 0x030A0000 || defined(HAVE_STDARG_PROTOTYPES)
454454
va_start(vargs, fmt);
455455
#else
456456
va_start(vargs);

0 commit comments

Comments
 (0)
0