Message-ID: <email address hidden>
Date: Sun, 13 Mar 2005 21:07:27 +0100
From: Adeodato =?iso-8859-1?Q?Sim=F3?= <email address hidden>
To: Stanislav Visnovsky <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: On-the-Fly spell checking causing kbabel to crash
Hello Stanislav,
we received in Debian a bug report claiming that KBabel crashed with a
certain .po file, and that it seemed to be related to on-the-fly spell
checking. You can read the full discussion at [1].
The crash is reproducible for me when going to the entry #98 in the
file mentioned in the report, being the spell checker language
English. I suspect the crash is triggered because that entry is a
little long, and as the checker language doesn't match the entry
language (English vs. Spanish), a lot of misspelling signals are
emitted.
I've been having a look this afternoon, and after a lot of struggling
I came to realice that using checkWord() instead of check() seems to
improve things. There is this comment in kbabel/kbhighlighting.cpp:
// yes I tried checkWord, the docs lie and it didn't give useful signals :-(
However, is more than 2 years old, and checkWord() seems to do fine
here. Perhaps you could considering making the change?
Also, it may be that kdelibs is at fault, too (as per the backtrace).
Sadly, I lack the knowledge to determine that, and anyway it seems
that applications should really use the proper function that the API
provides (checkWord, in this case).
Thanks for considering,
--
Adeodato Sim� EM: asp16 [ykwim] alu.ua.es | PK: DA6AE621
Listening to: Pet Shop Boys - You only tell me you love me when you're drunk
You cannot achieve the impossible without attempting the absurd.
Message-ID: <email address hidden> 1?Q?Sim= F3?= <email address hidden>
Date: Sun, 13 Mar 2005 21:07:27 +0100
From: Adeodato =?iso-8859-
To: Stanislav Visnovsky <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: On-the-Fly spell checking causing kbabel to crash
Hello Stanislav,
we received in Debian a bug report claiming that KBabel crashed with a
certain .po file, and that it seemed to be related to on-the-fly spell
checking. You can read the full discussion at [1].
[1] http:// bugs.debian. org/289646
The crash is reproducible for me when going to the entry #98 in the
file mentioned in the report, being the spell checker language
English. I suspect the crash is triggered because that entry is a
little long, and as the checker language doesn't match the entry
language (English vs. Spanish), a lot of misspelling signals are
emitted.
I've been having a look this afternoon, and after a lot of struggling kbhighlighting. cpp:
I came to realice that using checkWord() instead of check() seems to
improve things. There is this comment in kbabel/
// yes I tried checkWord, the docs lie and it didn't give useful signals :-(
However, is more than 2 years old, and checkWord() seems to do fine
here. Perhaps you could considering making the change?
Also, it may be that kdelibs is at fault, too (as per the backtrace).
Sadly, I lack the knowledge to determine that, and anyway it seems
that applications should really use the proper function that the API
provides (checkWord, in this case).
Thanks for considering,
--
Adeodato Sim� EM: asp16 [ykwim] alu.ua.es | PK: DA6AE621
Listening to: Pet Shop Boys - You only tell me you love me when you're drunk
You cannot achieve the impossible without attempting the absurd.