Le mardi 23 novembre 2010 à 08:49 -0600, Benjamin Peterson a écrit :
> 2010/11/23 Antoine Pitrou <solip...@pitrou.net>:
> > On Mon, 22 Nov 2010 22:00:08 -0600
> > Benjamin Peterson <benja...@python.org> wrote:
> >> 2010/11/22 Łukasz Langa <luk...@langa.pl>:
> >> > Wiadomość napisana przez Benjamin Peterson w dniu 2010-11-23, o godz. 
> >> > 00:47:
> >> >
> >> > No test?
> >> >
> >> >
> >> > The tests were there already, raising ResourceWarnings. After this 
> >> > change,
> >> > they stopped doing that. You may say: now they pass for the first time :)
> >>
> >> It looks like you added new API, though. For that, we would expect new 
> >> tests.
> >
> > It's an internal API, although ZipExtFile doesn't begin with an
> > underscore.
> 
> Why is it internal API then?

Because it's for use by ZipFile.open(). The ZipExtFile constructor is
not supposed to be called by the user.
You might instead asked why ZipExtFile isn't called _ZipExtFile instead,
and I have no idea.

Regards

Antoine.


_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to