From: Thomas Holst <Thomas_Holst@gmx.de>
To: emacs-orgmode@gnu.org
Subject: Re: org-capture regression?
Date: Thu, 20 Sep 2018 16:01:26 +0200 [thread overview]
Message-ID: <87musc46a1@Boerne> (raw)
In-Reply-To: <87sh24gwk0.fsf@nicolasgoaziou.fr> (Nicolas Goaziou's message of "Thu, 20 Sep 2018 14:53:03 +0200")
Hello,
· Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
> Thomas Holst <Thomas_Holst@gmx.de> writes:
>> [...snip...]
>> So here is I patch I came up with to re-enable theese features:
>
> Thank you. However, I have two issues with it.
>
> The first one is that I don't like `eval'. See, for example, this
> excerpt from the Elisp manual:
>
> On rare occasions, you may need to write code that evaluates a form
> that is computed at run time, such as after reading a form from text
> being edited or getting one from a property list. On these
> occasions, use the ‘eval’ function. Often ‘eval’ is not needed and
> something else should be used instead. For example, to get the value
> of a variable, while ‘eval’ works, ‘symbol-value’ is preferable; or
> rather than store expressions in a property list that then need to
> go through ‘eval’, it is better to store functions instead that are
> then passed to ‘funcall’.
>
> So, I'd rather use `symbol-value' and `funcall' and forbid arbitrary
> Sexps.
>
> Also, this introduces a discrepancy between :table-line-pos and other
> parameters, which do not allow variables nor functions. This is another
> issue.
>
> IOW, it may need be to be implemented at a lower level, with proper
> tooling.
>
> WDYT?
For me this sounds resonalble. Basically I am an elisp novice. I
will try to implement the feature like explaind above in
`org-capture-get'. So it can be used for other parameters.
When I have questions or a few lines of code I will come back here.
Right now I can live with my solution for my personal needs. Since my
time for working on this is limited it might take some time. It seems
that nobody else is missing this feature. So I will take the oportunity
and find my way through elisp.
--
Bis neulich ...
Thomas
prev parent reply other threads:[~2018-09-20 14:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-20 8:13 org-capture regression? Thomas Holst
2018-09-20 12:53 ` Nicolas Goaziou
2018-09-20 14:01 ` Thomas Holst [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87musc46a1@Boerne \
--to=thomas_holst@gmx.de \
--cc=emacs-orgmode@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.