[issue6490] os.popen documentation is probably wrong

A.M. Kuchling added the comment: Thanks for your patch! -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed title: os.popen documentation is probably wrong -> os.popen documentation is probably wrong ___ Python t

[issue6490] os.popen documentation is probably wrong

Roundup Robot added the comment: New changeset 3417a95df7e2 by Andrew Kuchling in branch 'default': #6490: Expand documentation for os.popen(). http://hg.python.org/cpython/rev/3417a95df7e2 -- nosy: +python-dev ___ Python tracker

[issue6490] os.popen documentation is probably wrong

Sam Kimbrel added the comment: Yes, I think that wording works a lot better. Thanks for the touch-up. -- ___ Python tracker ___ ___ Pyt

[issue6490] os.popen documentation is probably wrong

A.M. Kuchling added the comment: I thought the discussion of the return code was rather complicated and tried rewriting it; new patch attached. Is it an improvement? The new version also specifies the function's parameters. -- nosy: +akuchling Added file: http://bugs.python.org/file3

[issue6490] os.popen documentation is probably wrong

Sam Kimbrel added the comment: Updated the patch as per Martin's notes in msg203483. -- nosy: +sam.kimbrel Added file: http://bugs.python.org/file34896/6490-os-popen-docs.diff ___ Python tracker

[issue6490] os.popen documentation is probably wrong

Martin Panter added the comment: Also it would be good to document that it returns a text stream, not a binary stream. -- ___ Python tracker ___ _

[issue6490] os.popen documentation is probably wrong

Martin Panter added the comment: Please apply Neil Muller’s documentation patch. It is certainly better than the current state. If you want to improve it further, maybe get rid of the trailing comma, and mention that the close() method returns the exit status encoded like the wait() function.

[issue6490] os.popen documentation is probably wrong

Changes by Terry J. Reedy : -- title: os.popen documentation in 2.6 is probably wrong -> os.popen documentation is probably wrong ___ Python tracker ___