http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst
File Doc/library/zipfile.rst (right):

http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst#newcod...
Doc/library/zipfile.rst:210: is the name of the file in the archive, or a
:class:`ZipInfo` object.  The
If name the file in both cases, or can it refer to the Zipinfo object?  If it
always refers to the file, please reword that something like

"*name* is the name of the file in either the archive or a :class:`ZipInfo`
object.

If it name can be the ZipInfo object itself, then please reword something like:

"*name* can be either a :class:`ZipInfo` object or the name of a file within the
archive."

http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst#newcod...
Doc/library/zipfile.rst:247: Removed support of ``mode='U'``.
I would leave in the workaround information about using
:class:`io.TextIOWrapper`.  Maybe even show it as an example that does get
tested; something like:

f=io.TextIOWrapper(myarchive.open("myfile"))

except that I am not at all confident I have the syntax right -- which is why it
is worth documenting.

http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst
File Doc/library/zipfile.rst (right):

http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst#newcod...
Doc/library/zipfile.rst:210: is the name of the file in the archive, or a
:class:`ZipInfo` object.  The
On 2016/05/23 16:08:12, Jim.J.Jewett wrote:
> If name the file in both cases, or can it refer to the Zipinfo object?  If it
> always refers to the file, please reword that something like
> 
> "*name* is the name of the file in either the archive or a :class:`ZipInfo`
> object.
> 
> If it name can be the ZipInfo object itself, then please reword something
like:
> 
> "*name* can be either a :class:`ZipInfo` object or the name of a file within
the
> archive."

The latter. Thank you for your suggestion.

http://bugs.python.org/review/27029/diff/17248/Doc/library/zipfile.rst#newcod...
Doc/library/zipfile.rst:247: Removed support of ``mode='U'``.
On 2016/05/23 16:08:12, Jim.J.Jewett wrote:
> I would leave in the workaround information about using
> :class:`io.TextIOWrapper`.  Maybe even show it as an example that does get
> tested; something like:
> 
> f=io.TextIOWrapper(myarchive.open("myfile"))
> 
> except that I am not at all confident I have the syntax right -- which is why
it
> is worth documenting.

Done.