Review Request 121436: grepview: fix two crashes
Nicolai Hähnle
nhaehnle at gmail.com
Wed Dec 10 15:22:55 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121436/
-----------------------------------------------------------
Review request for KDevelop.
Repository: kdevplatform
Description
-------
The first crash is a crash-on-exit, when the kill() method of a no longer existing job is called. Guard againsts this crash by using a QPointer.
The second crash happens when GrepOutputDelegate::sizeHint() is called while using the combo-box of the grep output view to switch to previous search results. Simply change the code to be more defensive.
Diffs
-----
plugins/grepview/grepdialog.cpp bb00e47
plugins/grepview/grepoutputdelegate.cpp bcd911b
Diff: https://git.reviewboard.kde.org/r/121436/diff/
Testing
-------
manual testing / daily use
Thanks,
Nicolai Hähnle
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20141210/9a239895/attachment.html>
More information about the KDevelop-devel
mailing list