From 45d0b5cc44ffb6227a2379a39b00d480f253edd5 Mon Sep 17 00:00:00 2001 From: Raymond Hettinger Date: Mon, 12 Apr 2004 17:21:03 +0000 Subject: [PATCH] Use Py_RETURN_NONE macro where applicable. --- Objects/listobject.c | 22 ++++++++-------------- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/Objects/listobject.c b/Objects/listobject.c index 9c0afa91040..7a2cdeaf31f 100644 --- a/Objects/listobject.c +++ b/Objects/listobject.c @@ -652,20 +652,16 @@ listinsert(PyListObject *self, PyObject *args) PyObject *v; if (!PyArg_ParseTuple(args, "iO:insert", &i, &v)) return NULL; - if (ins1(self, i, v) == 0) { - Py_INCREF(Py_None); - return Py_None; - } + if (ins1(self, i, v) == 0) + Py_RETURN_NONE; return NULL; } static PyObject * listappend(PyListObject *self, PyObject *v) { - if (app1(self, v) == 0) { - Py_INCREF(Py_None); - return Py_None; - } + if (app1(self, v) == 0) + Py_RETURN_NONE; return NULL; } @@ -2089,8 +2085,7 @@ listreverse(PyListObject *self) { if (self->ob_size > 1) reverse_slice(self->ob_item, self->ob_item + self->ob_size); - Py_INCREF(Py_None); - return Py_None; + Py_RETURN_NONE; } int @@ -2190,10 +2185,9 @@ listremove(PyListObject *self, PyObject *v) int cmp = PyObject_RichCompareBool(self->ob_item[i], v, Py_EQ); if (cmp > 0) { if (list_ass_slice(self, i, i+1, - (PyObject *)NULL) != 0) - return NULL; - Py_INCREF(Py_None); - return Py_None; + (PyObject *)NULL) == 0) + Py_RETURN_NONE; + return NULL; } else if (cmp < 0) return NULL;