[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#289646: 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
  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.




Reply to: