This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author andymaier
Recipients andymaier, docs@python, eric.araujo, ezio.melotti, sandro.tosi, terry.reedy, tshepang
Date 2014-07-03.15:23:10
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1404400990.98.0.603566496709.issue14050@psf.upfronthosting.co.za>
In-reply-to
Content
Just out of curiosity: Why do the patches attached to this issue not have a "review" link?

Also, both (2.7 and 3.2) patches do not line up with the current 2.7 and 3.x tip, both hunks get rejected.

Comments on both patches:

1. It would be helpful if the text "Each item needs to define an ordering relationship." was followed by a statement about what happens when that is not the case (that is what Ezio also suggested), and where to look for details on how to define an ordering relationship. The problem with that is that there is no good place that is devoted to exactly that. The relatively best place for defining comparison I found so far is: https://docs.python.org/2.7/tutorial/datastructures.html#comparing-sequences-and-other-types and its 3.x equivalent.

2. The example that raises the TypeError is not needed, IMHO. This puts too much focus on the case that does not work. If we mention in the description that a TypeError is raised, that should be sufficient.

Andy
History
Date User Action Args
2014-07-03 15:23:11andymaiersetrecipients: + andymaier, terry.reedy, ezio.melotti, eric.araujo, sandro.tosi, docs@python, tshepang
2014-07-03 15:23:10andymaiersetmessageid: <1404400990.98.0.603566496709.issue14050@psf.upfronthosting.co.za>
2014-07-03 15:23:10andymaierlinkissue14050 messages
2014-07-03 15:23:10andymaiercreate