[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 “(”


reply via email to

[Prev in Thread] Current Thread [Next in Thread]