n00m wrote:
The second part of the compound if is backwards. So if this is headed
for production code, it better get fixed.
DaveA
Not sure I'm understanding your remark.
Well, others in the thread have observed the same thing, so maybe it
doesn't matter. But the quoted code had only one if statement:
>>def v_cmp(v1, v2):
>> if v1.x < v2.x and v1.y > v2.y:
>> return -1
>> return 0
And the first part of the compound if is a "<" comparison, while the
second part is a ">" comparison
This produces a different sort order than the default tuple comparison.
So it needs fixing.
DaveA
--
http://mail.python.org/mailman/listinfo/python-list
