diff --git a/Include/boolobject.h b/Include/boolobject.h index 7c4939b2396..aa5230a2e69 100644 --- a/Include/boolobject.h +++ b/Include/boolobject.h @@ -24,8 +24,8 @@ PyAPI_DATA(PyIntObject) _Py_ZeroStruct, _Py_TrueStruct; #define Py_True ((PyObject *) &_Py_TrueStruct) /* Macros for returning Py_True or Py_False, respectively */ -#define Py_RETURN_TRUE Py_INCREF(Py_True); return Py_True; -#define Py_RETURN_FALSE Py_INCREF(Py_False); return Py_False; +#define Py_RETURN_TRUE {Py_INCREF(Py_True); return Py_True;} +#define Py_RETURN_FALSE {Py_INCREF(Py_False); return Py_False;} /* Function to return a bool from a C long */ PyAPI_FUNC(PyObject *) PyBool_FromLong(long); diff --git a/Include/object.h b/Include/object.h index 8fdd4dc94e0..10334456f5c 100644 --- a/Include/object.h +++ b/Include/object.h @@ -634,7 +634,7 @@ PyAPI_DATA(PyObject) _Py_NoneStruct; /* Don't use this directly */ #define Py_None (&_Py_NoneStruct) /* Macro for returning Py_None from a function */ -#define Py_RETURN_NONE Py_INCREF(Py_None); return Py_None; +#define Py_RETURN_NONE {Py_INCREF(Py_None); return Py_None;} /* Py_NotImplemented is a singleton used to signal that an operation is diff --git a/Misc/NEWS b/Misc/NEWS index 87a82e97e2c..52ed529e2ee 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -155,6 +155,9 @@ Build C API ----- +- Added three new macros: Py_RETURN_NONE, Py_RETURN_TRUE, and Py_RETURN_FALSE. + Each return the singleton they mention after Py_INCREF()ing them. + - Added a new function, PyTuple_Pack(n, ...) for constructing tuples from a variable length argument list of Python objects without having to invoke the more complex machinery of Py_BuildValue(). PyTuple_Pack(3, a, b, c)