Re: [O] [PATCH] Support time units in est+

[Nick Dokos]

Malcolm Matalka <address@hidden> writes:

> Den 18 jan. 2017 22:19 skrev "Nick Dokos" <address@hidden>:
>
>     Malcolm Matalka <address@hidden> writes:
>    
>     > Hey, this is my first elisp programming so I'm quite certain this is a
>     > big hack.  I just stole elements from elsewhere in the file.  I'm hoping
>     > this is good enough to get accepted then perhaps someone with more taste
>     > would be able to refactor it to be a bit better.
>     >
>     > Let me know if I need to change anything.
>     >
>    
>     Yes, indeed: the first thing you should do is explain what you are
>     trying to do.  E.g. what does "est+" mean? Once you have explained
>     *what* you are trying to do, then somebody might be able to suggest
>     *how* to do it better (if it is a worthwhile thing to do in the first 
> place).
>
> I'm not sure I understand your question.  est+ is an existing functionality 
> in columnview that is
> documented. I've brought it closer to the regular effort functionality by 
> supporting time units.
>

My apologies: I didn't know about it.

-- 
Nick