Avoid potential crashes by QPointer technique
ClosedPublic

Authored by yurchor on Nov 4 2018, 5:18 PM.

Details

Summary

The last three EBN issues.

The technique is described here:
https://blogs.kde.org/node/3919

Some useful remarks can be found here:
https://phabricator.kde.org/D7285

Test Plan
  1. Try to start a new game and push the configure engine button. It should work as earlier. When trying to close the main window through DBus there should be no crashes.
  2. Emulate the pawn promotion somehow. The promotion window should be opened. Try to close the main window through DBus. There should be no crash.

Diff Detail

Repository
R425 Knights
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
yurchor requested review of this revision.Nov 4 2018, 5:18 PM
yurchor created this revision.
asemke accepted this revision.Nov 5 2018, 7:38 PM
This revision is now accepted and ready to land.Nov 5 2018, 7:38 PM
This revision was automatically updated to reflect the committed changes.