Angus> Sorry, JMarc. I've been lost in the depths of Windows Installer
Angus> land, for which I blame you entirely :-P
Angus> Looking at the patch, shouldn't updateSlot() be:
Angus> +void ControlSpellchecker::updateSlot(bool switched)
Angus> +{
Angus> + if (switched)
Angus> + hide();
Angus> + else
Angus> + update();
Angus> +}
This would mean that everytime updateSlot() is triggered (fairly
often, I guess), update() will be called and a new spell session will
start. Is that really what we want?
I will commit the patch as it is, although there is still the problem
that hide() eventually moves the cursor.
JMarc
