diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-12-04 08:15:20 +0100 |
|---|---|---|
| committer |  bstoeger <32835590+bstoeger@users.noreply.github.com> | 2019-12-04 08:37:14 +0100 |
| commit | 569e532f5c63f25abb81eec92937caec20f38d77 (patch) | |
| tree | 37779e2d473ae700cd31bf2fa0c42b4cbd39f82e /desktop-widgets | |
| parent | f0c61be7111c567358c6cb31b5fe83d5aafbe5c0 (diff) | |
| download | subsurface-569e532f5c63f25abb81eec92937caec20f38d77.tar.gz | |
Cleanup: Fix order of initialization in MainTab::MainTab()
For deterministic construction/destruction (i.e. objects are
destructed in reverse order of construction) it is crucial that
constructor initializer lists follow the order of the class
definition.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'desktop-widgets')
| -rw-r--r-- | desktop-widgets/tab-widgets/maintab.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/desktop-widgets/tab-widgets/maintab.cpp b/desktop-widgets/tab-widgets/maintab.cpp index 82e7e4fa5..194bf1c4f 100644 --- a/desktop-widgets/tab-widgets/maintab.cpp +++ b/desktop-widgets/tab-widgets/maintab.cpp @@ -50,11 +50,11 @@ struct Completers { MainTab::MainTab(QWidget *parent) : QTabWidget(parent), editMode(NONE), + modified(false), lastSelectedDive(true), lastTabSelectedDive(0), lastTabSelectedDiveTrip(0), - currentTrip(0), - modified(false) + currentTrip(0) { ui.setupUi(this); |