diff options
author | Katarina Behrens <Katarina.Behrens@cib.de> | 2016-06-29 16:21:52 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-06-30 06:57:43 +0000 |
commit | ec6956491a0c6553a9fb91f64c6ceb7cd8e8aa52 (patch) | |
tree | 8b012547377dd67fb643cf379b8f7ef2850421c7 /sc/source/ui | |
parent | 6f34143dfb061cb8dda76b9e4f449f1e6b4fc181 (diff) |
Step towards disposing of ScDataFormDlg properly
There were some VclPtrs hiding inside std::vector (twice), so fix
that, but the destructor is still not called (getting Window::~Window
inconsistency assert). With this patch however the screenshot cppunit
test no longer fails
Thanks noelgrandin for help
Change-Id: I8b46c4608cbab2070e36ef4abec226bcd997425e
Reviewed-on: https://gerrit.libreoffice.org/26772
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'sc/source/ui')
-rw-r--r-- | sc/source/ui/miscdlgs/datafdlg.cxx | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/sc/source/ui/miscdlgs/datafdlg.cxx b/sc/source/ui/miscdlgs/datafdlg.cxx index 64dd5de5e165..f1d8f636a18f 100644 --- a/sc/source/ui/miscdlgs/datafdlg.cxx +++ b/sc/source/ui/miscdlgs/datafdlg.cxx @@ -216,8 +216,14 @@ void ScDataFormDlg::dispose() m_pBtnNext.clear(); m_pBtnClose.clear(); m_pSlider.clear(); - m_pGrid.clear(); m_pFixedText.clear(); + for ( auto aFTIter = maFixedTexts.begin(); aFTIter != maFixedTexts.end(); ++aFTIter ) + aFTIter->disposeAndClear(); + for ( auto aEditIter = maEdits.begin(); aEditIter != maEdits.end(); ++aEditIter ) + aEditIter->disposeAndClear(); + maFixedTexts.clear(); + maEdits.clear(); + m_pGrid.clear(); ModalDialog::dispose(); } |