Re: [O] [PATCH] Add :sort option to clocktable report

2011-08-17 Thread Bastien
Hi Max,

Max Mikhanosha  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



[O] [PATCH] Add :sort option to clocktable report

2011-08-04 Thread Max Mikhanosha
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.

Perhaps this should be made a default? It seems clocktable report is
something that user would naturally expect to be sorted.



0016-Add-sort-option-to-clocktable-report.patch
Description: Binary data