[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: |
Michael Brand |
Subject: |
Re: [O] Add the capture feature "%(sexp)" to org-feed |
Date: |
Sun, 12 Aug 2012 11:37:07 +0200 |
Hi Bastien
On Sat, Aug 11, 2012 at 11:59 PM, Bastien <address@hidden> wrote:
> Unless I miss something obvious, the current version returns exactly
> the same thing than the one you propose -- can you double-check or let
> me know what is wrong with the current solution?
Your solution is wrong when there is a “%” not immediately followed by “(”:
On Sat, Aug 11, 2012 at 8:11 PM, Michael Brand
<address@hidden> wrote:
> On Sat, Aug 11, 2012 at 6:55 PM, Bastien <address@hidden> wrote:
>> Michael Brand <address@hidden> writes:
>>> [...]
>>> Also a small change against your latest patch to not stop at “%”
>>> without following “(” and possibly miss a “%(” before.
A realistic example, with point somewhere between “%” and the last “)”:
“- %(capitalize "5 % less (see item \"3)\" below)\n")\n”
This can happen with a feed template containing
"- %(capitalize \"%description\")\n"
and %description expanding to
“5 % less (see item "3)" below)\n”
All problems of the example are solved:
- escaping of the quotes that surround the “3)” in string
- seemingly unbalanced parenthesis in string coming from “3)”
- newline within %(...)
- “%” not immediately followed by “(”
Michael
- 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, 2012/08/09
- 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 <=
- 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
- Re: [O] Add the capture feature "%(sexp)" to org-feed, Bastien, 2012/08/10