Re: Ispell: Skipping part of text in texinfo-mode

[Arash Esbati <arash@gnu.org>]

Eli Zaretskii <eliz@gnu.org> writes:

> Then let's discuss them one by one, based on the Texinfo manual (which
> documents much more than you can find in our manuals).  Some commands
> need to ignore their arguments, some don't.

Will do so, no problem.  My first choice was by driven by auctex.texi
which I tested with.

> For example, @var's argument is more often than not be a real word or
> a phrase made out of words, like @var{next-file}.

I will remove 'var' from the skip list.

> It's definitely a possibility, I think, so we IMO should support it,
> at least as an option.

Hmm, we could say that people who want to use this must enable it with:

(add-hook 'texinfo-mode-hook (lambda ()
                               (setq ispell-parser 'texinfo)))

otherwise things will be as before.

> Doesn't ispell-check-comments only relevant to
> ispell-comments-and-strings?

Might be, then I still don't understand why my change instructs ispell
to spell-check comments.  Something is still off and I don't know what.

> Does this support nested markup, as in
>
>   @w{@code{(@var{file} . @var{buffer})}}
>
> in a way that will allow to spell-check "file" and "buffer"?  I'm not
> sure syntax-table feature supports nesting.

No, my code ignores everything inside @code{...}.  This is actually what
ispell does for .tex files.  The syntax-table stuff is to skip over
things like @kbd{C-c @key{@}} @key{RET}} robustly.

Best, Arash