[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH} Re: inline code block problem; first attempt
From: |
Nicolas Goaziou |
Subject: |
Re: [O] [PATCH} Re: inline code block problem; first attempt |
Date: |
Thu, 09 Jun 2016 13:41:38 +0200 |
Hello,
"Charles C. Berry" <address@hidden> writes:
> Patch atteched.
>
> Replaced ']' with '{' in the regex `header' group.
>
> Comments?
Thank you for the patch. This is better than what we have now. However,
I don't think this is the way to go.
Forbidding "{" is already a minor annoyance, but porting the same logic
to `org-element-inline-babel-call-parser', which suffers from the same
bug, means forbidding "(" ; this is a major one.
I'd rather do the same as the body part and allow "[" (and "{"
obviously), as long as they are paired. This is less restrictive, IMO,
even acceptable in the case of "(".
I also started a fix along those lines yesterday but didn't get time to
finish it yet.
WDYT?
Regards,
--
Nicolas Goaziou