Re: [AUCTeX-devel] Spell checking of macros

[Mosè Giordano]

Hi Arash,

2016-07-12 10:25 GMT+02:00 Arash Esbati <address@hidden>:
> Arash Esbati <address@hidden> writes:
>
>> I'm thinking about writing a function like this to catch this kind of
>> arguments:
>>
>>     (defun TeX-ispell-tex-arg-end (&optional arg1 arg2)
>>       (condition-case nil
>>           (progn
>>             (while (looking-at "[ \t\n]*\\[") (forward-sexp))
>>             (forward-sexp (or arg1 1))
>>             (while (looking-at "[ \t\n]*\\[") (forward-sexp))
>>             (forward-sexp (or arg2 1)))
>>         (error
>>          (message "Error skipping s-expressions at point %d." (point))
>>          (beep)
>>          (sit-for 2))))
>>
>> and do ("tabular[*xy]" TeX-ispell-tex-arg-end).
>
> Following up myself again, I implemented a function
> `TeX-ispell-tex-arg-end' to deal with constructs like:
>
>     \raisebox{2em}[1em][1em]{text} or
>     \begin{tabularx}{300pt}{|p{3cm}|>{\raggedright}X|c|S|} etc.
>
> I'd like to apply the attached patch.  I would appreciate any comments
> before doing this.

It looks overall good to me, but note that error messages *do not* end
with a period (see `error`'s docstring) and you should replace the
`message' function with `format' (and I don't think `beep' is needed).

Please, add documentation and mention the new feature in the news
section as well.

Bye,
Mosè