From 9169641b8b2995abb651826887773eb12cbf6064 Mon Sep 17 00:00:00 2001 From: Antoine Pitrou Date: Fri, 9 Jan 2009 22:12:30 +0000 Subject: [PATCH] Fix bug introduced in r68451: stdio must always be opened in line-buffered mode if isatty() is true. --- Python/pythonrun.c | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/Python/pythonrun.c b/Python/pythonrun.c index 8a1af3bdc5e..6819be55477 100644 --- a/Python/pythonrun.c +++ b/Python/pythonrun.c @@ -734,10 +734,10 @@ create_stdio(PyObject* io, int fd, int write_mode, char* name, char* encoding, char* errors) { - PyObject *buf = NULL, *stream = NULL, *text = NULL, *raw = NULL; + PyObject *buf = NULL, *stream = NULL, *text = NULL, *raw = NULL, *res; const char* mode; - const PyObject *line_buffering; - int buffering; + PyObject *line_buffering; + int buffering, isatty; if (Py_UnbufferedStdioFlag) buffering = 0; @@ -766,13 +766,21 @@ create_stdio(PyObject* io, text = PyUnicode_FromString(name); if (text == NULL || PyObject_SetAttrString(raw, "_name", text) < 0) goto error; - Py_CLEAR(raw); - Py_CLEAR(text); - - if (Py_UnbufferedStdioFlag) + res = PyObject_CallMethod(raw, "isatty", ""); + if (res == NULL) + goto error; + isatty = PyObject_IsTrue(res); + Py_DECREF(res); + if (isatty == -1) + goto error; + if (isatty || Py_UnbufferedStdioFlag) line_buffering = Py_True; else line_buffering = Py_False; + + Py_CLEAR(raw); + Py_CLEAR(text); + stream = PyObject_CallMethod(io, "TextIOWrapper", "OsssO", buf, encoding, errors, "\n", line_buffering);