GNU bug report logs - #32280
26.1; FLYSPELL-BUFFER sometimes misbehaves for some input in a large enough buffer

Previous Next

Full log

Message #8 received at 32280 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Artem Boldarev <artem.boldarev <at> gmail.com>
Cc: 32280 <at> debbugs.gnu.org
Subject: Re: bug#32280: 26.1;
 FLYSPELL-BUFFER sometimes misbehaves for some input in a large enough
 buffer
Date: Fri, 27 Jul 2018 15:45:27 +0300

> From: Artem Boldarev <artem.boldarev <at> gmail.com>
> Date: Thu, 26 Jul 2018 12:44:26 +0300
> 
> Checking large enough buffer with FlySpell leads to the unexpected
> results (at least, when spell checking Russian, but I believe that it is
> possible to reproduce the bug for other languages, at least Ukrainian).
> 
> For example, when checking large enough buffer (large enough to trigger
> flyspell-large-region) I got the following messages in the *Messages*
> buffer:
> 
> Local Ispell dictionary set to ru_RU
> Starting new Ispell process hunspell with ru_RU dictionary...
> Checking region...
> Spell Checking...100% [посимвольно]
> Spell Checking completed.
>   -> смом - 346: word not found
>   -> стостояния - 319: word not found
>   -> рекрсивного - 308: word not found
>   -> универсальнее - 266: word not found
>   -> генериует - 222: word not found
> 
> It was not able to find the misspelt words to highlight them in the 
> buffer which I tried
> to spell check. On the other hand, some not misspelt words were 
> highlighted (именно,
> бесконечный, усложняет).  Under other circumstances, these words are not 
> highlighted as misspelt (which is as it should be).

Can you post the text where this happens?

> The problem turned out to be in the flyspell-external-point-words: It
> makes some heuristic checks before calling (flyspell-word nil t). It
> seems that these checks are OK for English, as I never encountered any
> problems when spell-checking English texts.
> 
> Here is the version of the function which seems to be correct:

AFAICT, you have removed a single line:

			     (< found-length misspell-length)

Can you take me through your reasoning why this line is incorrect, and
what assumptions it made that are correct for English, but not for
Russian?

Thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.