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 zach.ware
Recipients brett.cannon, docs@python, marco.buttu, terry.reedy, zach.ware
Date 2016-12-04.05:30:31
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1480829432.41.0.321999910155.issue28860@psf.upfronthosting.co.za>
In-reply-to
Content
Indeed, thank you Marco for continuing this work!

The patches that I committed [1] were mostly done by Jelle Zijlstra.  There were not many tests that had dict ordering issues; but it looks like we used sorted() rather than list() in a few places.  I think using sorted() here is fine as well (though for the third one, I'd recommend turning the list comprehension into a generator expression argument to sorted, rather than sticking sorted() in the middle of the listcomp).

As far as why these patches are not compatible with Rietveld, they appear to have been generated by `diff` between two discrete files rather than by `hg diff` or `git diff`, and Reitveld can't find the files as named to be able to apply the patch.


[1] Viewable in aggregate here: https://hg.python.org/cpython/rev/769355e031:d0302d8ecbc1
History
Date User Action Args
2016-12-04 05:30:32zach.waresetrecipients: + zach.ware, brett.cannon, terry.reedy, docs@python, marco.buttu
2016-12-04 05:30:32zach.waresetmessageid: <1480829432.41.0.321999910155.issue28860@psf.upfronthosting.co.za>
2016-12-04 05:30:32zach.warelinkissue28860 messages
2016-12-04 05:30:31zach.warecreate