On Thu, 9 Jun 2016, Nicolas Goaziou wrote:
Hello,
"Charles C. Berry" <ccbe...@ucsd.edu> 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 "(".
Agreed.
I also started a fix along those lines yesterday but didn't get time to
finish it yet.
WDYT?
Sounds good.
Chuck
