Fix crashy dialogs (found by krazy and by hand)

Authored by croick on Jun 30 2017, 9:36 PM.

Description

Fix crashy dialogs (found by krazy and by hand)

Summary:

  • use a wrapper class which simplifies the use of QPointer for the creation of local QDialogs
  • some false positives are marked
  • some dialogs not identified by krazy have the main window as parent, they may still cause a crash, when the program is killed

Test Plan:
Before:

  • open "About KDevelop Platform" dialog
  • "killall kdevelop" -> kdevelop closes (as intended), but with crash

After:

  • open same dialog
  • "killall kdevelop" -> kdevelop silently quits

Reviewers: KDevelop, apol

Reviewed By: KDevelop, apol

Subscribers: apol, anthonyfieroni, kdevelop-devel

Differential Revision: https://phabricator.kde.org/D6308

Details

Committed
croickJun 30 2017, 9:57 PM
Reviewer
KDevelop
Differential Revision
D6308: Fix crashy dialogs (found by krazy and by hand)
Parents
R32:c6dd6e648481: Merge branch '5.1'
Branches
Unknown
Tags
Unknown