Skip to content

Commit

Permalink
MNT: always require va_start to have two arguments (#4820)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
tacaswell authored Jun 7, 2022
1 parent 3c0afd9 commit 4613ec0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Cython/Utility/MemoryView_C.c
Original file line number Diff line number Diff line change
Expand Up @@ -451,7 +451,7 @@ static void __pyx_fatalerror(const char *fmt, ...) Py_NO_RETURN {
va_list vargs;
char msg[200];

#ifdef HAVE_STDARG_PROTOTYPES
#if PY_VERSION_HEX >= 0x030A0000 || defined(HAVE_STDARG_PROTOTYPES)
va_start(vargs, fmt);
#else
va_start(vargs);
Expand Down

0 comments on commit 4613ec0

Please sign in to comment.