[REVIEW][3.5]fdo#46531, spell checking display fun

Caolán McNamara caolanm at redhat.com
Tue Feb 28 04:26:17 PST 2012


On Mon, 2012-02-27 at 17:07 +0100, Cedric Bosdonnat wrote:
> Hello people,
> 
> Could someone review and cherry-pick -s this commit into -3-5 branch?
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=adf45eced404c33be6db884a3e809725e7975872

FWIW, the nastiness in that SpellDialog is that there are a bunch of
things which would be natural to put directly into the ctor, but they
would require calling virtual methods during construction, which doesn't
work in c++, so there's some lame post-an-event hack and complete the
construction steps that want to call virtuals in the event-callback
which gets called on the next iteration of the main loop after the
construction has completed.

Dialog is fragile and tricky :-(

C.



More information about the LibreOffice mailing list