diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2014-01-26 17:21:00 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2014-01-26 17:21:00 (GMT) |
commit | 8003850e22788c9e02b8dea7076858e01aa24f65 (patch) | |
tree | 699b86530151ae88b9c0493f6d6e2225f3b0daf9 /Lib/codecs.py | |
parent | 714d7ffd0d652e187deb13b96aae7a1bbfcdecca (diff) | |
download | cpython-8003850e22788c9e02b8dea7076858e01aa24f65.zip cpython-8003850e22788c9e02b8dea7076858e01aa24f65.tar.gz cpython-8003850e22788c9e02b8dea7076858e01aa24f65.tar.bz2 |
Issue #8260: The read(), readline() and readlines() methods of
codecs.StreamReader returned incomplete data when were called after
readline() or read(size). Based on patch by Amaury Forgeot d'Arc.
Diffstat (limited to 'Lib/codecs.py')
-rw-r--r-- | Lib/codecs.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/Lib/codecs.py b/Lib/codecs.py index 6a6eb90..01ae0f3 100644 --- a/Lib/codecs.py +++ b/Lib/codecs.py @@ -463,15 +463,12 @@ class StreamReader(Codec): # read until we get the required number of characters (if available) while True: # can the request be satisfied from the character buffer? - if chars < 0: - if size < 0: - if self.charbuffer: - break - elif len(self.charbuffer) >= size: - break - else: + if chars >= 0: if len(self.charbuffer) >= chars: break + elif size >= 0: + if len(self.charbuffer) >= size: + break # we need more data if size < 0: newdata = self.stream.read() @@ -479,6 +476,8 @@ class StreamReader(Codec): newdata = self.stream.read(size) # decode bytes (those remaining from the last call included) data = self.bytebuffer + newdata + if not data: + break try: newchars, decodedbytes = self.decode(data, self.errors) except UnicodeDecodeError as exc: |