Hi Arash,

2016-07-08 19:11 GMT+02:00 Arash Esbati <esb...@gmx.de>:
> Hi Mosè,
>
> Mosè Giordano <m...@gnu.org> writes:
>
>> 2016-07-08 12:55 GMT+02:00 Arash Esbati <esb...@gmx.de>:
>>>
>>> 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è

_______________________________________________
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel

Reply via email to