diff options
| author | Guido van Rossum <guido@python.org> | 2001-01-08 01:26:47 (GMT) |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 2001-01-08 01:26:47 (GMT) |
| commit | 8628206b954541e1d47fe38f4c215ebbdb16baed (patch) | |
| tree | 64c796873b954bcd4bd6bd68cc289013d7624da2 /Objects/fileobject.c | |
| parent | 3a55f950a95f451e860190975ac43a163e48b820 (diff) | |
| download | cpython-8628206b954541e1d47fe38f4c215ebbdb16baed.zip cpython-8628206b954541e1d47fe38f4c215ebbdb16baed.tar.gz cpython-8628206b954541e1d47fe38f4c215ebbdb16baed.tar.bz2 | |
Let's hope that three time's a charm...
Tim discovered another "bug" in my get_line() code: while the comments
said that n<0 was invalid, it was in fact still called with n<0 (when
PyFile_GetLine() was called with n<0). In that case fortunately
executed the same code as for n==0.
Changed the comment to admit this fact, and changed Tim's MS speed
hack code to use 'n <= 0' as the criteria for the speed hack.
Diffstat (limited to 'Objects/fileobject.c')
| -rw-r--r-- | Objects/fileobject.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Objects/fileobject.c b/Objects/fileobject.c index 485c378..f13aa54 100644 --- a/Objects/fileobject.c +++ b/Objects/fileobject.c @@ -804,8 +804,7 @@ ms_getline_hack(FILE *fp) /* Internal routine to get a line. Size argument interpretation: > 0: max length; - = 0: read arbitrary line; - < 0: invalid + <= 0: read arbitrary line */ #ifdef HAVE_GETC_UNLOCKED @@ -828,7 +827,8 @@ get_line(PyFileObject *f, int n) PyObject *v; #ifdef USE_MS_GETLINE_HACK - if (n == 0) + + if (n <= 0) return ms_getline_hack(fp); #endif n2 = n > 0 ? n : 100; |