I've submitted a patch to the original bug On 21 August 2011 12:06, Lindsay Mathieson <[email protected]> wrote: > My patch for adding inline spell checking has a honking big bug, due > to my misunderstaning of QtPlatformPlugin behaviour, didn't realise it > unloaded the plugins in its destructor, so the following: > > bool TextCheckerClientQt::loadSpellChecker() > { > if (m_spellChecker) > return true; > > QtPlatformPlugin platformPlugin; > > if ((m_spellChecker = platformPlugin.createSpellChecker())) > return true; > > return false; > } > > crashes really badly :) It didn't show up in my testing as I used a > static plugin. > > Fixed it - should I just submit my patch for review against the first review? > > https://bugs.webkit.org/show_bug.cgi?id=44114 > > > -- > Lindsay >
-- Lindsay _______________________________________________ webkit-qt mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt
