Let's do as Guido says and return None instead of -1
This commit is contained in:
parent
af935e3bc4
commit
1607278c26
|
@ -387,7 +387,7 @@ stdprinter_write(PyStdPrinter_Object *self, PyObject *args)
|
|||
* I can't raise an exception here. It may lead to an
|
||||
* unlimited recursion in the case stderr is invalid.
|
||||
*/
|
||||
return PyLong_FromLong((long)-1);
|
||||
Py_RETURN_NONE;
|
||||
}
|
||||
|
||||
if (!PyArg_ParseTuple(args, "s", &c)) {
|
||||
|
@ -434,8 +434,7 @@ stdprinter_isatty(PyStdPrinter_Object *self)
|
|||
{
|
||||
long res;
|
||||
if (self->fd < 0) {
|
||||
Py_INCREF(Py_False);
|
||||
return Py_False;
|
||||
Py_RETURN_FALSE;
|
||||
}
|
||||
|
||||
Py_BEGIN_ALLOW_THREADS
|
||||
|
|
Loading…
Reference in New Issue