Jason R. Coombs jar...@jaraco.com added the comment:
Patch is also applicable to #6127.
--
nosy: +jaraco
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
Antoine Pitrou pit...@free.fr added the comment:
It looks like there is a disagreement between c-io and py-io as to what
the newline parameter in StringIO means.
I think the c-io version makes more sense (but I'm biased, since I wrote
it), while the py-io results are quite difficult to
Antoine Pitrou pit...@free.fr added the comment:
Can you test with the following patch?
--
keywords: +patch
Added file: http://bugs.python.org/file13610/memio.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
Antoine Pitrou pit...@free.fr added the comment:
Buildbots should be green for the release.
--
priority: - release blocker
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
Changes by Antoine Pitrou pit...@free.fr:
--
stage: - patch review
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
___
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
I confirmed the patch works.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
Antoine Pitrou pit...@free.fr added the comment:
Committed in r71151, thanks!
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
I'm sure, but with following test code,
def run(module):
print(///)
print(//, module)
memio = module.StringIO(newline=None)
# The C StringIO decodes newlines in write() calls, but the
Changes by Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp:
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5645
___
___