diff options
| author | Mark Page <aptitude@btconnect.com> | 2016-06-07 09:09:07 +0100 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-06-08 10:59:32 +0000 |
| commit | 5164ac456f3cb51949fe3bec293660fab74d26de (patch) | |
| tree | 8f30c614d496113f3d515d2301179680291699fc /sfx2 | |
| parent | aed4c42491b294cb57efc950a4d0209116be6452 (diff) | |
tdf#96099 Remove SfxInterfaceArr_Impl typedef
Added comment in SfxModule with warning about using smart pointers
Change-Id: I1997d41a0a77c7e82e486bcdaee0ea876fdc61ad
Reviewed-on: https://gerrit.libreoffice.org/25999
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/appl/module.cxx | 4 | ||||
| -rw-r--r-- | sfx2/source/control/msgpool.cxx | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/sfx2/source/appl/module.cxx b/sfx2/source/appl/module.cxx index bb1730dedcde..397872375122 100644 --- a/sfx2/source/appl/module.cxx +++ b/sfx2/source/appl/module.cxx @@ -92,7 +92,8 @@ ImageList* SfxModule_Impl::GetImageList( ResMgr* pResMgr, bool bBig ) rpList = new ImageList(); } - return rpList; } + return rpList; +} SFX_IMPL_SUPERCLASS_INTERFACE(SfxModule, SfxShell) @@ -147,7 +148,6 @@ SfxModule::~SfxModule() break; } } - } delete pImpl; diff --git a/sfx2/source/control/msgpool.cxx b/sfx2/source/control/msgpool.cxx index eb38323424f6..ab832d6464c7 100644 --- a/sfx2/source/control/msgpool.cxx +++ b/sfx2/source/control/msgpool.cxx @@ -55,7 +55,7 @@ void SfxSlotPool::RegisterInterface( SfxInterface& rInterface ) { // add to the list of SfxObjectInterface instances if(!_pInterfaces) - _pInterfaces.reset(new SfxInterfaceArr_Impl); + _pInterfaces.reset(new std::vector<SfxInterface*> ); _pInterfaces->push_back(&rInterface); // Stop at a (single) Null-slot (for syntactic reasons the interfaces @@ -106,7 +106,7 @@ void SfxSlotPool::ReleaseInterface( SfxInterface& rInterface ) return ; // remove from the list of SfxInterface instances - SfxInterfaceArr_Impl::iterator i = std::find(_pInterfaces->begin(), _pInterfaces->end(), &rInterface); + auto i = std::find(_pInterfaces->begin(), _pInterfaces->end(), &rInterface); if(i != _pInterfaces->end()) _pInterfaces->erase(i); } |