diff options
author | Neal Norwitz <nnorwitz@gmail.com> | 2002-01-01 19:07:13 (GMT) |
---|---|---|
committer | Neal Norwitz <nnorwitz@gmail.com> | 2002-01-01 19:07:13 (GMT) |
commit | 649b75954a1880e8a6dc15066a3041bfabac959a (patch) | |
tree | 33eb8a29a81dfa31e48ba8966df71cee1de8345f | |
parent | a6e975801e37bef27deb3207f6fc8f0c59affb46 (diff) | |
download | cpython-649b75954a1880e8a6dc15066a3041bfabac959a.zip cpython-649b75954a1880e8a6dc15066a3041bfabac959a.tar.gz cpython-649b75954a1880e8a6dc15066a3041bfabac959a.tar.bz2 |
SF Patch #494863, file.xreadlines() should raise ValueError if file is closed
This makes xreadlines behave like all other file methods
(other than close() which just returns).
-rw-r--r-- | Misc/NEWS | 4 | ||||
-rw-r--r-- | Objects/fileobject.c | 2 |
2 files changed, 6 insertions, 0 deletions
@@ -6,6 +6,10 @@ Type/class unification and new-style classes Core and builtins +- file.xreadlines() now raises a ValueError if the file is closed: + Previously, an xreadlines object was returned which would raise + a ValueError when the xreadlines.next() method was called. + Extension modules Library diff --git a/Objects/fileobject.c b/Objects/fileobject.c index 27612f4..9af5ca9 100644 --- a/Objects/fileobject.c +++ b/Objects/fileobject.c @@ -1025,6 +1025,8 @@ file_xreadlines(PyFileObject *f) { static PyObject* xreadlines_function = NULL; + if (f->f_fp == NULL) + return err_closed(); if (!xreadlines_function) { PyObject *xreadlines_module = PyImport_ImportModule("xreadlines"); |