[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] Separate clocksum format for durations >= 1 day
From: |
Toby Cubitt |
Subject: |
Re: [O] [PATCH] Separate clocksum format for durations >= 1 day |
Date: |
Tue, 20 Nov 2012 18:28:19 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Nov 20, 2012 at 11:12:10AM -0500, Mike McLean wrote:
> On Sat, Nov 17, 2012 at 11:02 AM, Toby Cubitt <address@hidden> wrote:
>
> > On Sat, Nov 17, 2012 at 03:42:24PM +0100, Nicolas Goaziou wrote:
> > > Toby Cubitt <address@hidden> writes:
> > >
> > > > I've replaced the cons cells with additional plist properties, as you
> > > > suggested. The resulting customization ui still isn't wonderful in my
> > > > opinion. But it does the job, and I'm not sure how much scope there is
> > > > for improving it further. If you see a way, by all means feel free to
> > > > make the changes yourself. I really don't mind what format you go with
> > > > for org-time-clocksum-format, as long as it supports the new formatting
> > > > features implemented in the patch.
> > >
> > > Considering I'm not an expert in customize ui, it's good enough as it
> > > is.
> >
> > OK. If someone thinks of a way to improve the customization ui (keeping
> > the same data type), it's easy to change it later without breaking
> > anything.
> >
> > > I've applied your patch (with some small changes in a docstring). Thank
> > > you again for all that work.
> >
> > Glad we finally found a good implementation. Thanks to you for all your
> > helpful feedback.
> >
> >
> I like the new implementation and customization options, but now when I do
> ~C-c C-x C-c~ to get a sub-tree time, I get nothing but zeros (~0d 0:00~)
> for every subtree.
Argh. This is a bug in org-hours-to-clocksum-string, which doesn't
truncate the computed number of minutes to an integer.
Instead of changing org-hours-to-clocksum-string, it's probably
preferable to make org-minutes-to-clocksum-string more robust, so it
copes correctly with floating point arguments.
The attached patch does this. Sorry for letting this slip through.
Toby
--
Dr T. S. Cubitt
Mathematics and Quantum Information group
Department of Mathematics
Complutense University
Madrid, Spain
email: address@hidden
web: www.dr-qubit.org
0001-Fix-org-minutes-to-clocksum-string-to-cope-with-floa.patch
Description: Text Data
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, (continued)
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Nicolas Goaziou, 2012/11/14
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/14
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Nicolas Goaziou, 2012/11/14
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/14
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/16
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Nicolas Goaziou, 2012/11/17
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/17
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Nicolas Goaziou, 2012/11/17
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/17
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Mike McLean, 2012/11/20
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day,
Toby Cubitt <=
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Nicolas Goaziou, 2012/11/20
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Mike McLean, 2012/11/21
- [O] [bug] Separate clocksum format for durations >= 1 day, Sebastien Vauban, 2012/11/30
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Achim Gratz, 2012/11/06
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Toby Cubitt, 2012/11/06
- Re: [O] [PATCH] Separate clocksum format for durations >= 1 day, Achim Gratz, 2012/11/06