Wei-Cheng Pan added the comment:

> The patch needs tests and documentation.

Ok, I'll update it later.

>> +    parser.add_argument('--no-ensure-ascii', action='store_true', 
>> default=False,
>I'd go with ``action='store_false', default=True``.

If I'm not misreading your comment, this will change the original behavior, 
right? (because options.no_ensure_ascii will be set to True by default)

----------

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

Reply via email to