[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] Add the capture feature "%(sexp)" to org-feed
From: |
Bastien |
Subject: |
Re: [O] Add the capture feature "%(sexp)" to org-feed |
Date: |
Thu, 09 Aug 2012 16:53:10 +0200 |
User-agent: |
Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.1.50 (gnu/linux) |
Hi Michael,
Michael Brand <address@hidden> writes:
> You omitted the Local variables sentence-end-double-space: t. Isn't
> this a good idea for all the users like me that have set this to nil
> in their config? I wanted to add it to more files as soon as I will
> change them.
I've check against current Emacs trunk, and only three files/packages
uses this: rst.el, filesets.el and mh-e.
So I'd rather not pollute the hundred files of Org with this.
> “The current date.” is probably meant to remain “the current date.”.
Yes.
>> specifically, I used `org-at-regexp-p' instead of
>> `org-capture-inside-embedded-elisp-p'.
>
> With `org-capture-inside-embedded-elisp-p' I wanted to be quite more
> waterproof than `org-at-regexp-p' with the new additional closing
> parenthesis in `org-capture-template-embedded-elisp-re'. See its
> comment “to deal with for example %(length ")")”. Isn't it a good idea
> to use `forward-sexp' in Emacs Lisp Mode for this? What can I improve?
(org-at-regexp-p org-capture-template-embedded-elisp-re) already
returns the correct value for %(length ")") -- but you're right there
is a problem with my patch: `org-at-regexp-p' does not match over
multiple lines. Maybe you can play with `org-in-regexp':
(org-in-regexp org-capture-template-embedded-elisp-re 3)
If using `forward-sexp' is necessary let's do so -- but I thought it
was too complex first.
Thanks,
--
Bastien
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/07
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/09
- Re: [O] Add the capture feature "%(sexp)" to org-feed,
Bastien <=
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/09
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/10
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/11
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/11
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/11
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/11
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/12
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/12
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Ivan Andrus, 2012/08/09
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Michael Brand, 2012/08/10