Hi Arash, 2016-07-12 10:25 GMT+02:00 Arash Esbati <esb...@gmx.de>: > Arash Esbati <esb...@gmx.de> 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è _______________________________________________ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel