[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] Support time units in est+
From: |
Nick Dokos |
Subject: |
Re: [O] [PATCH] Support time units in est+ |
Date: |
Thu, 19 Jan 2017 10:44:37 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
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