On Mar 13, 2012 9:34 AM, "Hyrum K Wright" <hyrum.wright_at_wandisco.com> wrote:
>
> On Tue, Mar 13, 2012 at 1:09 AM, Greg Stein <gstein_at_gmail.com> wrote:
> > On Tue, Mar 13, 2012 at 00:55, <hwright_at_apache.org> wrote:
> >>...
> >> +++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Tue Mar
13 04:55:26 2012
> >> @@ -36,6 +36,7 @@ import optparse
> >> import xml
> >> import urllib
> >> import logging
> >> +import cStringIO as StringIO
> >
> > No need to rename this module, as is typical with back-compat importing.
>
> This isn't backward compat, it's an alternative implementation that is
> a drop-in replacement for the StringIO module, hence the desire to
> alias it.
Dude. I *know* what it is. My point is that it is silly to rename it when
it has been a standard module for *years*. Nobody really uses StringIO any
more. It is all cStringIO, possibly renamed thru import logic that is used
for back-compat. The rename is superfluous gunk nowadays.
-g
Received on 2012-03-13 14:46:12 CET