[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [AUCTeX-devel] Extending xparse.el to parse argument specification
From: |
Mosè Giordano |
Subject: |
Re: [AUCTeX-devel] Extending xparse.el to parse argument specification |
Date: |
Sat, 8 Oct 2016 15:15:59 +0200 |
Hi Arash,
I'm sorry for the very late replay, I finally found some time to look
to the patch. I don't have major comments, looks but I warmly suggest
you to write some comprehensive tests so that we won't break anything
that is working.
Thank you!
Mosè
2016-09-28 22:30 GMT+02:00 Mosè Giordano <address@hidden>:
> Hi Arash,
>
> 2016-09-22 12:30 GMT+02:00 Arash Esbati <address@hidden>:
>> Hi all,
>>
>> I was thinking about how to extend xparse.el in order to parse argument
>> specs (which can be quite complicated) and now have a working setup --
>> diff and patched xparse.el are attached.
>
> Wow, this is a great news, parsing macros created with xparse has
> always been in my wishlist, but I've never found the time (or the
> boldness) to actually undertake the task!
>
>> Main work is done by
>> `LaTeX-xparse-macro-parse'. I don't have much experience with this kind
>> of parsing within Emacs. Question: Is this kind of implementation
>> robust enough or are there better ways to do the job?
>
> I couldn't look closely to the patch, but looking to your past
> contributions I'm confident you did a good job :-)
>
> My understanding is that \DeclareDocumentCommand syntax can be fairly
> wild, I'm not sure Emacs can always get it right. I'll check out your
> contribution when I'll find some time (I hope in one or two weeks, I'm
> pretty busy lately), if your style file works in most common cases I
> think we can include it in AUCTeX.
>
> Bye,
> Mosè
- Re: [AUCTeX-devel] Extending xparse.el to parse argument specification,
Mosè Giordano <=