[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] Fix clocktable scope parameter
From: |
Nicolas Goaziou |
Subject: |
Re: [O] [PATCH] Fix clocktable scope parameter |
Date: |
Sun, 05 Mar 2017 11:54:59 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hello,
Thank you. Some comments follow.
Eduardo Bellani <address@hidden> writes:
> org-clock.el: Fix clocktable scope parameter
Why is it a "fix"? I think this patch belongs to master branch, not
maint.
> * lisp/org-clock.el (org-dblock-write:clocktable): Funcall the scope
> argument if it is a function.
>
> * doc/org.texi: Document the feature of using a function, both bounded
> and as a lambda form, as the scope for the clocktable.
Since a bounded symbol and a lambda form are both functions, I'd rather
not distinguish them. Actually, this distinction is usually not made.
> + symbol @r{scan the files returned by calling the nullary
> function bound to this symbol}
> + lambda-form @r{Same as above, but the nullary function is
> passed as a lambda form.}
See above. I suggest:
function @r{...}
> - ((pred consp) scope)
> + ((pred #'functionp) (funcall scope))
(pred functionp)
> + ((pred listp) scope)
Is there any reason to change consp into listp? This change doesn't
appear in the commit message.
Could you provide an entry in ORG-NEWS? Also, would you mind writing
a couple of simple tests in test-org-clock.el?
Regards,
--
Nicolas Goaziou