Alexander Belopolsky <[EMAIL PROTECTED]> added the comment:

With respect to range_eq8_normalize2.patch, it is unusual to have a 
function that consumes a reference to its argument.  I would combine 
normalize_stop with PyNumber_Index and make it similar to validate_step 
with respect to reference counting.

Note that if you choose stop = start + len*step normaization, you will 
not need to create 'one' in normalize_stop.

With your patch I see

>>> range(0,6,2)
range(0, 6, 2)
>>> range(0,5,2)
range(0, 5, 2)

I would expect one of these ranges normalized.

__________________________________
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue2603>
__________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to