[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: Request for advice on making spell check ignore quoted text
From:       Ingo =?iso-8859-15?q?Kl=F6cker?= <ingo.kloecker () epost ! de>
Date:       2002-01-30 23:26:39
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Thursday 31 January 2002 00:05, Dave Corrie wrote:
> If no one else is working on this, then I'd like to make a
> modification to KMail that would stop the spell checker from checking
> quoted text in the compose window (ie lines that start with "> " or
> whatever indentation you have set in your preferences).

Nice idea. AFAIK nobody is working on this. But please note that we are 
finally in a feature freeze for KDE 3.0. Your modifications won't be 
added to KMail before development for KDE 3.1 begins. But of course 
this shouldn't stop you from implementing it.

> 1) Create a copy of the buffer which excluded quoted lines and call
> KSpell::check().
>
> 2) Create a QStringList of words, which excludes the words on quoted
> lines and call KSpell::checkList().
>
> 3) Call KSpell::checkWord() with every word not on a quoted line.
>
[snip]
> #3 would need much less memory, but will "sacrifice speed when
> checking large numbers of words" according to the KSpell docs.
>
> It comes down to the usual memory/cpu trade off. My personal
> preference is #1, but I'd be interested to hear other peoples'
> opinions.

I don't think that #3 would be too bad with regard to speed. After all 
in general an email message doesn't contain too many words. IMO this 
speed issue is only valid for documents which contain tens of thousand 
words.

> Are there any planned KMail developments which would swing the
> balance in one direction?

Not afaik.

> And finally, would there need to be a user preference to turn off
> this functionality or would it be better to have it always on (thus
> avoiding clutter in the prefs pages)?

No. This feature really doesn't need to be turned off.

Regards,
Ingo
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE8WIExGnR+RTDgudgRApkmAJ9zDmrSD16bfEt49gsctSc4qfRBMgCghQaH
3ob5qX6/Mf68N5zy2xMIY5o=
=RigQ
-----END PGP SIGNATURE-----
_______________________________________________
kmail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail
[prev in list] [next in list] [prev in thread] [next in thread]