Mailing List Archive: Python: Dev

Re: [Python-checkins] cpython (merge 3.2 -> default): Make TextIOWrapper's documentation clearer by copying the newline argument's

Index | Next | Previous | View Threaded

chris.jerdonek at gmail

Aug 4, 2012, 8:41 AM

Post #1 of 3 (185 views)
Permalink

Re: [Python-checkins] cpython (merge 3.2 -> default): Make TextIOWrapper's documentation clearer by copying the newline argument's

On Fri, Aug 3, 2012 at 3:59 PM, antoine.pitrou
<python-checkins [at] python> wrote:
> http://hg.python.org/cpython/rev/f17a1410ebe5
> changeset: 78401:f17a1410ebe5
> summary:
> Make TextIOWrapper's documentation clearer by copying the newline argument's description from open().

Now that this change is made, it may make sense to update the
subprocess documentation to reference TextIOWrapper's documentation
instead of open()'s (since use of the 'U' flag to open() is
discouraged in new code).

"All line endings will be converted to '\n' as described for the
universal newlines 'U' mode argument to open()."

(from http://docs.python.org/dev/library/subprocess.html#frequently-used-arguments
)

--Chris
_______________________________________________
Python-Dev mailing list
Python-Dev [at] python
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/list-python-dev%40lists.gossamer-threads.com

victor.stinner at gmail

Aug 4, 2012, 11:51 AM

Post #2 of 3 (175 views)
Permalink

Re: [Python-checkins] cpython (merge 3.2 -> default): Make TextIOWrapper's documentation clearer by copying the newline argument's [In reply to]

2012/8/4 Chris Jerdonek <chris.jerdonek [at] gmail>:
> On Fri, Aug 3, 2012 at 3:59 PM, antoine.pitrou
> <python-checkins [at] python> wrote:
>> http://hg.python.org/cpython/rev/f17a1410ebe5
>> changeset: 78401:f17a1410ebe5
>> summary:
>> Make TextIOWrapper's documentation clearer by copying the newline argument's description from open().
>
> Now that this change is made, it may make sense to update the
> subprocess documentation to reference TextIOWrapper's documentation
> instead of open()'s (since use of the 'U' flag to open() is
> discouraged in new code).
>
> "All line endings will be converted to '\n' as described for the
> universal newlines 'U' mode argument to open()."
>
> (from http://docs.python.org/dev/library/subprocess.html#frequently-used-arguments
> )

Good idea, can you please open an issue? The documentation is wrong:
UTF-8 is not used, it's the locale encoding.

Victor
_______________________________________________
Python-Dev mailing list
Python-Dev [at] python
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/list-python-dev%40lists.gossamer-threads.com

chris.jerdonek at gmail

Aug 5, 2012, 12:56 PM

Post #3 of 3 (164 views)
Permalink

Re: [Python-checkins] cpython (merge 3.2 -> default): Make TextIOWrapper's documentation clearer by copying the newline argument's [In reply to]

On Sat, Aug 4, 2012 at 11:51 AM, Victor Stinner
<victor.stinner [at] gmail> wrote:
> 2012/8/4 Chris Jerdonek <chris.jerdonek [at] gmail>:
>> Now that this change is made, it may make sense to update the
>> subprocess documentation to reference TextIOWrapper's documentation
>> instead of open()'s (since use of the 'U' flag to open() is
>> discouraged in new code).
>
> Good idea, can you please open an issue? The documentation is wrong:
> UTF-8 is not used, it's the locale encoding.

I created an issue for this (with patch) here:

http://bugs.python.org/issue15561

--Chris
_______________________________________________
Python-Dev mailing list
Python-Dev [at] python
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/list-python-dev%40lists.gossamer-threads.com

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads