Message132146
Thanks for the patch.
Are there practical cases where the operator.index check is more useful that the isinstance(..., (int, long)) check that you originally proposed (off-tracker)? While I agree that it's the right fix in principle, the operator.index check smells a bit like new functionality to me, and risks changing behaviour for those (rare, I hope) user-defined classes that implement both __index__ and sequence methods. |
|
Date |
User |
Action |
Args |
2011-03-25 20:10:15 | mark.dickinson | set | recipients:
+ mark.dickinson, Robert.Kern |
2011-03-25 20:10:15 | mark.dickinson | set | messageid: <1301083815.9.0.890146212027.issue11673@psf.upfronthosting.co.za> |
2011-03-25 20:10:15 | mark.dickinson | link | issue11673 messages |
2011-03-25 20:10:15 | mark.dickinson | create | |
|