[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] Add :sort option to clocktable report
From: |
Bastien |
Subject: |
Re: [O] [PATCH] Add :sort option to clocktable report |
Date: |
Wed, 17 Aug 2011 12:16:34 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) |
Hi Max,
Max Mikhanosha <address@hidden> writes:
> Attached patch adds new options :sort to the clocktable report
> options. Valid values:
>
> time-up or T - highest time on top
> time-down - lowest time on top
>
> For multi-file clock reports, entries in each file are sorted
> separately and then the files are also sorted based on file total.
That's a *great* addition. I've tested the patch and it works well.
As the patch is more than 15 lines, you would need to sign the FSF
copyright assignment before I can apply the patch. I'll send you the
relevant papers in private.
> Perhaps this should be made a default? It seems clocktable report is
> something that user would naturally expect to be sorted.
Yes, I think :sort time-up should be the default.
Best,
--
Bastien