<div dir="ltr">Please update the docstring in subprocess.py with the wording improvements that you settle on while you're at it.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jul 6, 2013 at 6:03 AM, Ronald Oussoren <span dir="ltr"><<a href="mailto:ronaldoussoren@mac.com" target="_blank">ronaldoussoren@mac.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On 6 Jul, 2013, at 14:09, Ronald Oussoren <<a href="mailto:ronaldoussoren@mac.com">ronaldoussoren@mac.com</a>> wrote:<br>
<br>
><br>
> On 6 Jul, 2013, at 13:59, R. David Murray <<a href="mailto:rdmurray@bitdance.com">rdmurray@bitdance.com</a>> wrote:<br>
>><br>
</div><div class="im">>> IMO, either the default should be mentioned first, or the default<br>
>> should be mentioned in a parenthetical.  Otherwise it sounds like<br>
>> newline translation is being done in both modes.  Logically that makes<br>
>> no sense, so the above construction will likely lead to, at a minimum,<br>
>> an interruption in the flow for the reader, and at worse even more<br>
>> confusion than not mentioning it at all.<br>
><br>
> You've got a point there. Converting the next text (", otherwise ...") to a parententical<br>
> seems to be the cleanest fix, creating a separate sentence for the ``False`` case introduces<br>
> duplication unless I restructure the text.<br>
<br>
</div>I didn't like the parenthentical after all. Would this work for you?:<br>
<br>
<br>
-   If *universal_newlines* is ``True``, the file objects *stdin*, *stdout* and<br>
-   *stderr* will be opened as text streams in :term:`universal newlines` mode<br>
+   If *universal_newlines* is ``False`` the file objects *stdin*, *stdout* and<br>
+   *stderr* will be opened as binary streams, and no line ending conversion is done.<br>
+<br>
+   If *universal_newlines* is ``True``, these file objects<br>
<div class="im">+   will be opened as text streams in :term:`universal newlines` mode<br>
    using the encoding returned by :func:`locale.getpreferredencoding(False)<br>
</div><div class="im">-   <locale.getpreferredencoding>`, otherwise these streams will be opened<br>
-   as binary streams.  For *stdin*, line ending characters<br>
+   <locale.getpreferredencoding>`.  For *stdin*, line ending characters<br>
    ``'\n'`` in the input will be converted to the default line separator<br>
    :data:`os.linesep`.  For *stdout* and *stderr*, all line endings in the<br>
    output will be converted to ``'\n'``.  For more information see the<br>
<br>
</div>That is, a new paragraph is added before the existing one to explain the behavior of<br>
"not universal_newlines".<br>
<br>
Ronald<br>
<div class="im"><br>
<br>
><br>
> Ronald<br>
><br>
>><br>
>> --David<br>
>> _______________________________________________<br>
>> Python-Dev mailing list<br>
>> <a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
>> <a href="http://mail.python.org/mailman/listinfo/python-dev" target="_blank">http://mail.python.org/mailman/listinfo/python-dev</a><br>
>> Unsubscribe: <a href="http://mail.python.org/mailman/options/python-dev/ronaldoussoren%40mac.com" target="_blank">http://mail.python.org/mailman/options/python-dev/ronaldoussoren%40mac.com</a><br>
><br>
> _______________________________________________<br>
> Python-Dev mailing list<br>
> <a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
> <a href="http://mail.python.org/mailman/listinfo/python-dev" target="_blank">http://mail.python.org/mailman/listinfo/python-dev</a><br>
> Unsubscribe: <a href="http://mail.python.org/mailman/options/python-dev/ronaldoussoren%40mac.com" target="_blank">http://mail.python.org/mailman/options/python-dev/ronaldoussoren%40mac.com</a><br>
<br>
_______________________________________________<br>
Python-Dev mailing list<br>
<a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
<a href="http://mail.python.org/mailman/listinfo/python-dev" target="_blank">http://mail.python.org/mailman/listinfo/python-dev</a><br>
</div>Unsubscribe: <a href="http://mail.python.org/mailman/options/python-dev/greg%40krypto.org" target="_blank">http://mail.python.org/mailman/options/python-dev/greg%40krypto.org</a><br>
</blockquote></div><br></div>