Review Request: Fix crash when KTextEdit::Factory is not found

Robert Mathias Marmorstein robert at narnia.homeunix.com
Sat Mar 26 07:56:38 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100949/
-----------------------------------------------------------

Review request for KDevelop and Quanta.


Summary
-------

This patch fixes a crash which occurs when closing kdevelop or quanta after the "KTextEditor::Factory not found" error box is displayed.  The problem is a simple logic error caused by the fact that QApplicationCore::exit() does not immediately abort an application.  Instead, it causes event processing to cease, which indirectly terminates the application AFTER the function has returned to the main event loop.


Diffs
-----

  shell/partcontroller.cpp 5a81de0 

Diff: http://git.reviewboard.kde.org/r/100949/diff


Testing
-------

Recompiled, ran quanta and kdevelop.  Error message dialog still comes up (as it should), but the program closes without crashing now.


Thanks,

Robert Mathias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110326/27e72a93/attachment.html>


More information about the KDevelop-devel mailing list