auctex-devel
[Top][All Lists]

## Re: [AUCTeX-devel] Spell checking of macros

 From: Mosè Giordano Subject: Re: [AUCTeX-devel] Spell checking of macros Date: Sat, 9 Jul 2016 15:57:50 +0200

Hi Arash,

2016-07-08 19:11 GMT+02:00 Arash Esbati <address@hidden>:
> Hi Mosè,
>
>
>> 2016-07-08 12:55 GMT+02:00 Arash Esbati <address@hidden>:
>>>
>>> Following up myself, I think I have a working setup for an AUCTeX ispell
>>> dictionary (diff attached).  I hope this way, it will be easy to add
>>> more macro/env names.  I would appreciate any comments.  After that, I
>>> will update the documentation and make it available.
>
> Thanks for looking at this.
>
>> Thanks for your work!  Some questions:
>>
>> * why all those eval-when-compile'?
>
> The final regexp which goes into byte-compiled file is built with:
>
> --8<---------------cut here---------------start------------->8---
> (defvar TeX-ispell-skip-cmds-one-arg-regexp
>   (eval-when-compile
>     (concat "\\\\"
>             (regexp-opt (TeX-ispell-sort-skip-cmds-list 1) t)))
>   "Regexp of LaTeX commands with one argument to be skipped.")
> --8<---------------cut here---------------end--------------->8---

Ok, then why this regexp is built with eval-when-compile'?

> I needed the other eval-when-compile' to make the compiler happy.
> Byte-compile tex-ispell.el' and have a look at the .elc file.
>
>> * why you decided to add all elements of TeX-ispell-skip-cmds-list'
>> in tex-ispell.tex', instead of adding the appropriate macros in each
>> style file?  Performance?
>
> I think on the long run with lots of macros, it will help.  I did it
> mainly because I wanted to feed regexp-opt' with all the macros and
> have one regexp to do the work.  I started with the idea to have macros
> in the respective style files, but the one-file-approach seems easier to
> maintain.  Parsed elements can still be added by styles with
> TeX-ispell-skip-set(car|cdr)'.

Fine, thanks.

>> * is is possible to skip the second
>> argument of "\begin{}{}"?  For example. in the first example you
>> showed us in this thread ispell' still wants to check "llr".
>
> Yes it is.  It works for me for tabularx & tabulary with (already in
> the last diff):
>
>   (TeX-ispell-skip-setcdr
>     '(...
>       ("tabular[xy]" ispell-tex-arg-end 2)))
>
> I see that I have missed \begin{tabular*} (since I never use it ;-),
> should be easily fixed with:
>
>   (TeX-ispell-skip-setcdr
>     '(...
>       ("tabular[*xy]" ispell-tex-arg-end 2)))

Maybe the regexp should be "tabular[*xy]?", right?

Bye,
Mosè

`