From 9d95bc58e82c65527a364906eca740151fe3301b Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Mon, 22 Oct 2012 08:30:32 +0200 Subject: De-inline some destructors in QtWidgets MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Destructors should be out-of-line so that the compiler doesn't generate one per translation unit. Apart from creating more work for the compiler, it can also lead to duplicated vtables if the dtor is the first virtual function (reimplementation), and all other virtuals are inline, too. Duplicate vtables then break RTTI. In addition, having virtual dtors de-inlined allows us to add code to them in a BC way. As a final argument, this change may lead to less code app-side, since a sequence of cross-DLL calls (to member variable dtors) is replaced by a single cross-DLL call to the new out-of-line dtor. Change-Id: Ifb8c4aa992c75d61ba9ac8de5ab41d1e96b0a0b1 Reviewed-by: Stephen Kelly Reviewed-by: Friedemann Kleint Reviewed-by: Jędrzej Nowacki --- src/widgets/dialogs/qwizard.cpp | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src/widgets/dialogs/qwizard.cpp') diff --git a/src/widgets/dialogs/qwizard.cpp b/src/widgets/dialogs/qwizard.cpp index 7f1cb09225..44518ec7d5 100644 --- a/src/widgets/dialogs/qwizard.cpp +++ b/src/widgets/dialogs/qwizard.cpp @@ -3383,6 +3383,13 @@ QWizardPage::QWizardPage(QWidget *parent) connect(this, SIGNAL(completeChanged()), this, SLOT(_q_updateCachedCompleteState())); } +/*! + Destructor. +*/ +QWizardPage::~QWizardPage() +{ +} + /*! \property QWizardPage::title \brief the title of the page -- cgit v1.2.3