RetroSearch Browse

Sat Mar 26 20:19:09 CET 2011 · https://mail.python.org/pipermail/python-dev/2011-March/110162.html

Am 26.03.2011 20:00, schrieb Terry Reedy:
> On 3/26/2011 2:17 PM, Georg Brandl wrote:
>> "Refactor" doesn't sound like it belongs in the 3.1 branch...
> 
>>> -        for i in range(len(pattern)):
>>> -            c = pattern[i]
>>> +        for i, c in enumerate(pattern):
> 
> I would call thin 'Replace obsolete idiom in' rather than 'Refactor'.
> So are you criticizing the replacement or the mislabeling?

No - I believe he is critizing that a stylistic change is done
in a maintenance branch. It's not a bug fix, AFAICT, so it should not
have been done.

Regards,
Martin

P.S. I haven't looked into the specific context, but the diff alone
may actually cause behavior changes, depending on what pattern exactly
is.

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://mail.python.org/pipermail/python-dev/2011-March/110162.html below:

[Python-Dev] cpython (3.1): #2650: Refactor re.escape to use enumerate().