[Psi-Devel] [Patches] [Patch] Spell checker doesn't handle abbreviated words
mblsha at gmail.com
Sun Jul 13 14:32:51 PDT 2008
On 29.06.2008, at 1:09, Norman Rasmussen wrote:
> There's no point in even generating a file to attach for this one.
> This patch allows for quotes within words (need to avoid spell
> checking words starting and ending with quotes).
> My logic on removing the \b's is that the way regex defines and
> greedily matches \w's implicitly means that there must be \b's
> around it. Also the way that Psi's highlighting logic works, the
> matching starts searching after the end of the last match, so
> matching within a word is avoided.
Could you please give some examples?
> Also this forces words to be at least two characters before they'll
> be subject to the spell checker, which seems fine because as far as
> I can tell, (with aspell at least), all one letter words are 'valid'.
Seems to be true with Mac OS X's spell checker too.
One other problem that you approach doesn't fix is that aspell
erroneously highlights numbers. And there's one more problem as well:
we shouldn't highlight words on top of which the cursor is located.
I have a private prototype patch that fixes two problems I've
specified, and I wanted to clean it up for inclusion in 0.13.
More information about the Psi-Devel