Tim> If you mean you changed it to: Tim> static PyObject* silly(PyObject* obj, PyObject *args) Tim> { Tim> if (1) Tim> { Tim> Py_DECREF(Py_None); Tim> } Tim> then I would say that's the right thing to do - all the time, in Tim> all code. Then you also agree we should bracify the Py_*REF macros, right? <wink> Skip