From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Brand <michael.ch.brand@gmail.com>
Subject: Re: Add the capture feature "%(sexp)" to org-feed
Date: Thu, 9 Aug 2012 20:00:37 +0200
Message-ID: <CALn3zohvUBi4emSi8Wm1J7cVb5Wj6mvsVfqyvdTNWei+OnyKEw@mail.gmail.com>
References: <michael.ch.brand@gmail.com>
	<CALn3zojPR+yudDsmJxX4Q_CjahwOi7uAOOw6ehOHQoB1CL4=Pg@mail.gmail.com>
	<7822.1339898820@alphaville>
	<CALn3zoipejd1=f_KR3V0qAphReU6P5nWj0jiy56MmdG0hT4njw@mail.gmail.com>
	<87y5lqe21l.fsf@gnu.org>
	<CALn3zogGT5Lr2Fj4JdGQ8Q3S+Ry7NY5mQAdQTJ2FQZxc=0m3Bg@mail.gmail.com>
	<87liho86cp.fsf@gnu.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([208.118.235.92]:47339)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <michael.ch.brand@gmail.com>) id 1SzX2L-00032K-4g
	for emacs-orgmode@gnu.org; Thu, 09 Aug 2012 14:00:52 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <michael.ch.brand@gmail.com>) id 1SzX2E-0000Pp-NQ
	for emacs-orgmode@gnu.org; Thu, 09 Aug 2012 14:00:45 -0400
In-Reply-To: <87liho86cp.fsf@gnu.org>
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
To: Bastien <bzg@gnu.org>
Cc: Org Mode <emacs-orgmode@gnu.org>

Hi Bastien

On Thu, Aug 9, 2012 at 4:53 PM, Bastien <bzg@gnu.org> wrote:

> 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.

That's all right.

> (org-at-regexp-p org-capture-template-embedded-elisp-re) already
> returns the correct value for %(length ")")

For the above example yes, but not with more than one %():
=93- %(capitalize "simple percent-escape")\n- %(capitalize "one more")=94

Also non-greedy regexp is not enough here:
=93%(capitalize "(some) text")=94

> -- 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.

When I wrote the patch my conclusion was that regexp alone is not
sufficient. For real feed templates used by me, not only for
artificial examples. I must admit that my solution with `forward-sexp'
is complicated but it makes available the power of the Emacs Lisp Mode
parser. Maybe there are still some simplifications?

Michael