Differential D711

On shutdown make sure the ApplicationModel goes first
ClosedPublic
Actions

Authored by ervin on Dec 27 2015, 9:31 PM.

Details

Reviewers
bensi
franckarrecot
Commits
R4:04991bc7044e: On shutdown make sure the ApplicationModel goes first
Summary

This is necessary when everything is integrated, on shutdown lists are
emptied which makes the GUI react and ask for more information which is
unwanted at that point. Make sure we cut the links between model and GUI
before the GUI is destroyed further.

This fixes a crash on exit which showed up if you had a project selected
for instance and closed the window.

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin updated this revision to Diff 1642.Dec 27 2015, 9:31 PM
ervin retitled this revision from to On shutdown make sure the ApplicationModel goes first.
ervin updated this object.
ervin edited the test plan for this revision. (Show Details)
ervin added reviewers: bensi, franckarrecot.
bensi accepted this revision.Dec 28 2015, 10:54 AM
bensi edited edge metadata.
This revision is now accepted and ready to land.Dec 28 2015, 10:54 AM
Closed by commit R4:04991bc7044e: On shutdown make sure the ApplicationModel goes first (authored by ervin). · Explain WhyDec 28 2015, 8:59 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 1657

View Options

src/widgets/applicationcomponents.h

Loading...
View Options

src/widgets/applicationcomponents.cpp

Loading...