Re: [PATCH] Separate clocksum format for durations >= 1 day

[Toby Cubitt <tsc25@cantab.net>]

On Mon, Nov 05, 2012 at 11:47:12AM +0100, Achim Gratz wrote:
> Nicolas Goaziou writes:
> > As you suggest, I think a better plan is to replace all of them with
> > a single `org-time-clocksum-display-function'. Its expected value would
> > be a function accepting 2 arguments: hours and minutes, as numbers and
> > it should return a string.
> 
> Actually, it seems an even better plan IMHO to just have one single
> function that takes a format string and the total time in minutes or
> seconds and allow customization of the format string.

A format string isn't sufficient. It requires the number of time
components (days, hours, minutes, etc.) to be fixed in advance. Whereas a
function can decide whether to display e.g. days+hours or hours+minutes
depending on whether the time is longer or shorter than 24h.

A defcustom that can either be a format string *or* a function might be
an option.

Or maybe I've misunderstood what you're proposing?

Toby
-- 
Dr T. S. Cubitt
Mathematics and Quantum Information group
Department of Mathematics
Complutense University
Madrid, Spain

email: tsc25@cantab.net
web:   www.dr-qubit.org