Serhiy Storchaka added the comment:

Well, good.

But I still have several nitpicks. Here is revised patch.

* Now ASCII encoding is used to test list() output. And tests now run even if 
sys.stdout is a StringIO.
* test_list_verbose now test that printed files are actually separated by one 
new line, not by just spaces and not by two new lines.
* safe_print was simplified and renamed to _safe_print. Now streams without the 
"encoding" attribute are supported.
* Minor style fixes.

----------
Added file: 
http://bugs.python.org/file33792/fix_tarfile_list_print_lone_surrogate_v5.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue19920>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to