Raymond Hettinger added the comment:

Nick, the latest version of the patch looks like a correct solution.

Before applying, please add a block comment showing why these shenanigans are 
necessary (i.e. the use self.__lt__ instead of the less-than operator because 
the former doesn't fall into recursive operator flipping).

Also, please add a note to the docs indicating 1) that NotImplemented is now 
supported as of version 3.4 and 2) that when speed matters, it is always better 
to code all four ordering operators by hand rather than paying the cost of 
total_ordering's layers of indirection.

----------
assignee: rhettinger -> ncoghlan

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue10042>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to