[PATCH 17 of 24] churn: sort users with same churn by name

Bryan O'Sullivan bos at serpentine.com
Mon Dec 17 15:02:14 CST 2012


On Sun, Dec 16, 2012 at 2:34 PM, Mads Kiilerich <mads at kiilerich.com> wrote:

> churn: sort users with same churn by name
>

I like the intent of this change.


> This makes the output order well-defined and improves code readability.
>
> -    sortkey = ((not opts.get('sort')) and (lambda x: -sum(x[1])) or None)
>

Improves? Hardly. That is impenetrable.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20121217/85652a99/attachment.html>


More information about the Mercurial-devel mailing list