[Libreoffice-commits] core.git: 2 commits - desktop/source sd/source
Noel Grandin
noel.grandin at collabora.co.uk
Mon Jun 25 07:13:00 UTC 2018
desktop/source/deployment/gui/dp_gui_extlistbox.cxx | 12 ++++-----
desktop/source/deployment/gui/dp_gui_extlistbox.hxx | 4 +--
desktop/source/deployment/gui/dp_gui_updatedialog.cxx | 22 ++++++----------
desktop/source/deployment/gui/dp_gui_updatedialog.hxx | 4 +--
sd/source/ui/animations/CustomAnimationDialog.cxx | 24 ++++++++----------
sd/source/ui/animations/CustomAnimationDialog.hxx | 6 ++--
sd/source/ui/animations/CustomAnimationPane.cxx | 12 ++++-----
sd/source/ui/animations/CustomAnimationPane.hxx | 2 -
8 files changed, 39 insertions(+), 47 deletions(-)
New commits:
commit e5dfdbcc2084bfdb213afbe03d00136e93d5c4d2
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Thu Jun 21 15:34:16 2018 +0200
loplugin:useuniqueptr in CustomAnimationDialog
Change-Id: Ic2e93c1493159c9602bd68b409052117766dcc02
Reviewed-on: https://gerrit.libreoffice.org/56328
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/sd/source/ui/animations/CustomAnimationDialog.cxx b/sd/source/ui/animations/CustomAnimationDialog.cxx
index 224bf1223bc1..fbcdad9097c9 100644
--- a/sd/source/ui/animations/CustomAnimationDialog.cxx
+++ b/sd/source/ui/animations/CustomAnimationDialog.cxx
@@ -2167,9 +2167,9 @@ CustomAnimationDialog::CustomAnimationDialog(vcl::Window* pParent, STLPropertySe
sal_uInt16 nTimingId = mpTabControl->GetPageId("timing");
sal_uInt16 nTextAnimId = mpTabControl->GetPageId("textanim");
- mpEffectTabPage = VclPtr<CustomAnimationEffectTabPage>::Create( mpTabControl, mpSet );
+ mpEffectTabPage = VclPtr<CustomAnimationEffectTabPage>::Create( mpTabControl, mpSet.get() );
mpTabControl->SetTabPage( nEffectId, mpEffectTabPage );
- mpDurationTabPage = VclPtr<CustomAnimationDurationTabPage>::Create( mpTabControl, mpSet );
+ mpDurationTabPage = VclPtr<CustomAnimationDurationTabPage>::Create( mpTabControl, mpSet.get() );
mpTabControl->SetTabPage( nTimingId, mpDurationTabPage );
bool bHasText = false;
@@ -2178,7 +2178,7 @@ CustomAnimationDialog::CustomAnimationDialog(vcl::Window* pParent, STLPropertySe
if( bHasText )
{
- mpTextAnimTabPage = VclPtr<CustomAnimationTextAnimTabPage>::Create( mpTabControl, mpSet );
+ mpTextAnimTabPage = VclPtr<CustomAnimationTextAnimTabPage>::Create( mpTabControl, mpSet.get() );
mpTabControl->SetTabPage( nTextAnimId, mpTextAnimTabPage );
}
else
@@ -2202,8 +2202,8 @@ void CustomAnimationDialog::dispose()
mpDurationTabPage.disposeAndClear();
mpTextAnimTabPage.disposeAndClear();
- delete mpSet;
- delete mpResultSet;
+ mpSet.reset();
+ mpResultSet.reset();
mpTabControl.clear();
TabDialog::dispose();
@@ -2211,23 +2211,21 @@ void CustomAnimationDialog::dispose()
STLPropertySet* CustomAnimationDialog::getResultSet()
{
- delete mpResultSet;
-
mpResultSet = createDefaultSet();
- mpEffectTabPage->update( mpResultSet );
- mpDurationTabPage->update( mpResultSet );
+ mpEffectTabPage->update( mpResultSet.get() );
+ mpDurationTabPage->update( mpResultSet.get() );
if( mpTextAnimTabPage )
- mpTextAnimTabPage->update( mpResultSet );
+ mpTextAnimTabPage->update( mpResultSet.get() );
- return mpResultSet;
+ return mpResultSet.get();
}
-STLPropertySet* CustomAnimationDialog::createDefaultSet()
+std::unique_ptr<STLPropertySet> CustomAnimationDialog::createDefaultSet()
{
Any aEmpty;
- STLPropertySet* pSet = new STLPropertySet();
+ std::unique_ptr<STLPropertySet> pSet(new STLPropertySet());
pSet->setPropertyDefaultValue( nHandleMaxParaDepth, makeAny( sal_Int32(-1) ) );
pSet->setPropertyDefaultValue( nHandleHasAfterEffect, makeAny( false ) );
diff --git a/sd/source/ui/animations/CustomAnimationDialog.hxx b/sd/source/ui/animations/CustomAnimationDialog.hxx
index 2ad30fb2bcb1..14f605b2984f 100644
--- a/sd/source/ui/animations/CustomAnimationDialog.hxx
+++ b/sd/source/ui/animations/CustomAnimationDialog.hxx
@@ -148,11 +148,11 @@ public:
STLPropertySet* getResultSet();
- static STLPropertySet* createDefaultSet();
+ static std::unique_ptr<STLPropertySet> createDefaultSet();
private:
- STLPropertySet* mpSet;
- STLPropertySet* mpResultSet;
+ std::unique_ptr<STLPropertySet> mpSet;
+ std::unique_ptr<STLPropertySet> mpResultSet;
VclPtr<TabControl> mpTabControl;
diff --git a/sd/source/ui/animations/CustomAnimationPane.cxx b/sd/source/ui/animations/CustomAnimationPane.cxx
index 2421d51ec461..a3a3d5d44dc3 100644
--- a/sd/source/ui/animations/CustomAnimationPane.cxx
+++ b/sd/source/ui/animations/CustomAnimationPane.cxx
@@ -917,7 +917,7 @@ void CustomAnimationPane::UpdateLook()
}
}
-void addValue( STLPropertySet* pSet, sal_Int32 nHandle, const Any& rValue )
+void addValue( std::unique_ptr<STLPropertySet>& pSet, sal_Int32 nHandle, const Any& rValue )
{
switch( pSet->getPropertyState( nHandle ) )
{
@@ -1122,9 +1122,9 @@ static bool hasVisibleShape( const Reference< XShape >& xShape )
return true;
}
-STLPropertySet* CustomAnimationPane::createSelectionSet()
+std::unique_ptr<STLPropertySet> CustomAnimationPane::createSelectionSet()
{
- STLPropertySet* pSet = CustomAnimationDialog::createDefaultSet();
+ std::unique_ptr<STLPropertySet> pSet = CustomAnimationDialog::createDefaultSet();
pSet->setPropertyValue( nHandleCurrentPage, makeAny( mxCurrentPage ) );
@@ -1666,13 +1666,13 @@ void CustomAnimationPane::changeSelection( STLPropertySet const * pResultSet, ST
void CustomAnimationPane::showOptions(const OString& sPage)
{
- STLPropertySet* pSet = createSelectionSet();
+ std::unique_ptr<STLPropertySet> pSet = createSelectionSet();
- VclPtrInstance< CustomAnimationDialog > pDlg(this, pSet, sPage);
+ VclPtrInstance< CustomAnimationDialog > pDlg(this, pSet.get(), sPage);
if( pDlg->Execute() )
{
addUndo();
- changeSelection( pDlg->getResultSet(), pSet );
+ changeSelection( pDlg->getResultSet(), pSet.get() );
updateControls();
}
}
diff --git a/sd/source/ui/animations/CustomAnimationPane.hxx b/sd/source/ui/animations/CustomAnimationPane.hxx
index 086b717dbc84..f1aff2e25cdc 100644
--- a/sd/source/ui/animations/CustomAnimationPane.hxx
+++ b/sd/source/ui/animations/CustomAnimationPane.hxx
@@ -109,7 +109,7 @@ private:
void moveSelection( bool bUp );
void onPreview( bool bForcePreview );
- STLPropertySet* createSelectionSet();
+ std::unique_ptr<STLPropertySet> createSelectionSet();
void changeSelection( STLPropertySet const * pResultSet, STLPropertySet const * pOldSet );
static css::uno::Any getProperty1Value( sal_Int32 nType, const CustomAnimationEffectPtr& pEffect );
commit b9426828aefad95aace7f8935ef5dbd6a4664091
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Thu Jun 21 10:45:32 2018 +0200
loplugin:useuniqueptr in desktop
Change-Id: Iff29d7d5962b441678c91bcd0319ac07c6488b34
Reviewed-on: https://gerrit.libreoffice.org/56327
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/desktop/source/deployment/gui/dp_gui_extlistbox.cxx b/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
index 60bc41d003e5..5df9f5574093 100644
--- a/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
+++ b/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
@@ -224,8 +224,8 @@ void ExtensionBox_Impl::Init()
m_xRemoveListener = new ExtensionRemovedListener( this );
- m_pLocale = new lang::Locale( Application::GetSettings().GetLanguageTag().getLocale() );
- m_pCollator = new CollatorWrapper( ::comphelper::getProcessComponentContext() );
+ m_pLocale.reset( new lang::Locale( Application::GetSettings().GetLanguageTag().getLocale() ) );
+ m_pCollator.reset( new CollatorWrapper( ::comphelper::getProcessComponentContext() ) );
m_pCollator->loadDefaultCollator( *m_pLocale, i18n::CollatorOptions::CollatorOptions_IGNORE_CASE );
Show();
@@ -256,8 +256,8 @@ void ExtensionBox_Impl::dispose()
m_xRemoveListener.clear();
- delete m_pLocale;
- delete m_pCollator;
+ m_pLocale.reset();
+ m_pCollator.reset();
::svt::IExtensionListBox::dispose();
}
@@ -841,7 +841,7 @@ bool ExtensionBox_Impl::FindEntryPos( const TEntry_Impl& rEntry, const long nSta
if ( nStart == nEnd )
{
- eCompare = rEntry->CompareTo( m_pCollator, m_vEntries[ nStart ] );
+ eCompare = rEntry->CompareTo( m_pCollator.get(), m_vEntries[ nStart ] );
if ( eCompare < 0 )
return false;
else if ( eCompare == 0 )
@@ -862,7 +862,7 @@ bool ExtensionBox_Impl::FindEntryPos( const TEntry_Impl& rEntry, const long nSta
}
const long nMid = nStart + ( ( nEnd - nStart ) / 2 );
- eCompare = rEntry->CompareTo( m_pCollator, m_vEntries[ nMid ] );
+ eCompare = rEntry->CompareTo( m_pCollator.get(), m_vEntries[ nMid ] );
if ( eCompare < 0 )
return FindEntryPos( rEntry, nStart, nMid-1, nPos );
diff --git a/desktop/source/deployment/gui/dp_gui_extlistbox.hxx b/desktop/source/deployment/gui/dp_gui_extlistbox.hxx
index 8a2ede5822ca..356a4ff2e238 100644
--- a/desktop/source/deployment/gui/dp_gui_extlistbox.hxx
+++ b/desktop/source/deployment/gui/dp_gui_extlistbox.hxx
@@ -140,8 +140,8 @@ class ExtensionBox_Impl : public ::svt::IExtensionListBox
std::vector< TEntry_Impl > m_vEntries;
std::vector< TEntry_Impl > m_vRemovedEntries;
- css::lang::Locale *m_pLocale;
- CollatorWrapper *m_pCollator;
+ std::unique_ptr<css::lang::Locale> m_pLocale;
+ std::unique_ptr<CollatorWrapper> m_pCollator;
//Holds weak references to extensions to which is we have added an XEventListener
std::vector< css::uno::WeakReference<
diff --git a/desktop/source/deployment/gui/dp_gui_updatedialog.cxx b/desktop/source/deployment/gui/dp_gui_updatedialog.cxx
index 37513af9e031..852addaaa829 100644
--- a/desktop/source/deployment/gui/dp_gui_updatedialog.cxx
+++ b/desktop/source/deployment/gui/dp_gui_updatedialog.cxx
@@ -548,14 +548,8 @@ void UpdateDialog::dispose()
{
storeIgnoredUpdates();
- for (auto const& listboxEntry : m_ListboxEntries)
- {
- delete listboxEntry;
- }
- for (auto const& ignoredUpdate : m_ignoredUpdates)
- {
- delete ignoredUpdate;
- }
+ m_ListboxEntries.clear();
+ m_ignoredUpdates.clear();
m_pUpdates.disposeAndClear();
m_pchecking.clear();
m_pthrobber.clear();
@@ -719,7 +713,7 @@ void UpdateDialog::addEnabledUpdate( OUString const & name,
UpdateDialog::Index *pEntry = new UpdateDialog::Index( ENABLED_UPDATE, nIndex, name );
m_enabledUpdates.push_back( data );
- m_ListboxEntries.push_back( pEntry );
+ m_ListboxEntries.emplace_back( pEntry );
if ( ! isIgnoredUpdate( pEntry ) )
{
@@ -742,7 +736,7 @@ void UpdateDialog::addDisabledUpdate( UpdateDialog::DisabledUpdate const & data
UpdateDialog::Index *pEntry = new UpdateDialog::Index( DISABLED_UPDATE, nIndex, data.name );
m_disabledUpdates.push_back( data );
- m_ListboxEntries.push_back( pEntry );
+ m_ListboxEntries.emplace_back( pEntry );
isIgnoredUpdate( pEntry );
addAdditional( pEntry, SvLBoxButtonKind::DisabledCheckbox );
@@ -755,7 +749,7 @@ void UpdateDialog::addSpecificError( UpdateDialog::SpecificError const & data )
UpdateDialog::Index *pEntry = new UpdateDialog::Index( SPECIFIC_ERROR, nIndex, data.name );
m_specificErrors.push_back( data );
- m_ListboxEntries.push_back( pEntry );
+ m_ListboxEntries.emplace_back( pEntry );
addAdditional( pEntry, SvLBoxButtonKind::StaticImage);
}
@@ -976,7 +970,7 @@ void UpdateDialog::getIgnoredUpdates()
uno::Any aPropValue( uno::Reference< beans::XPropertySet >( xNameAccess->getByName( aIdentifier ), uno::UNO_QUERY_THROW )->getPropertyValue( PROPERTY_VERSION ) );
aPropValue >>= aVersion;
IgnoredUpdate *pData = new IgnoredUpdate( aIdentifier, aVersion );
- m_ignoredUpdates.push_back( pData );
+ m_ignoredUpdates.emplace_back( pData );
}
}
@@ -1106,7 +1100,7 @@ void UpdateDialog::setIgnoredUpdate( UpdateDialog::Index const *pIndex, bool bIg
if ( bIgnore && !bFound )
{
IgnoredUpdate *pData = new IgnoredUpdate( aExtensionID, aVersion );
- m_ignoredUpdates.push_back( pData );
+ m_ignoredUpdates.emplace_back( pData );
}
}
}
@@ -1222,7 +1216,7 @@ IMPL_LINK_NOARG(UpdateDialog, allHandler, CheckBox&, void)
for (auto const& listboxEntry : m_ListboxEntries)
{
if ( listboxEntry->m_bIgnored || ( listboxEntry->m_eKind != ENABLED_UPDATE ) )
- insertItem( listboxEntry, SvLBoxButtonKind::DisabledCheckbox );
+ insertItem( listboxEntry.get(), SvLBoxButtonKind::DisabledCheckbox );
}
}
else
diff --git a/desktop/source/deployment/gui/dp_gui_updatedialog.hxx b/desktop/source/deployment/gui/dp_gui_updatedialog.hxx
index 5e83051f9ddd..6116ca95ee71 100644
--- a/desktop/source/deployment/gui/dp_gui_updatedialog.hxx
+++ b/desktop/source/deployment/gui/dp_gui_updatedialog.hxx
@@ -194,8 +194,8 @@ private:
std::vector< dp_gui::UpdateData > m_enabledUpdates;
std::vector< UpdateDialog::DisabledUpdate > m_disabledUpdates;
std::vector< UpdateDialog::SpecificError > m_specificErrors;
- std::vector< UpdateDialog::IgnoredUpdate* > m_ignoredUpdates;
- std::vector< Index* > m_ListboxEntries;
+ std::vector< std::unique_ptr<UpdateDialog::IgnoredUpdate> > m_ignoredUpdates;
+ std::vector< std::unique_ptr<Index> > m_ListboxEntries;
std::vector< dp_gui::UpdateData > & m_updateData;
rtl::Reference< UpdateDialog::Thread > m_thread;
css::uno::Reference< css::deployment::XExtensionManager > m_xExtensionManager;
More information about the Libreoffice-commits
mailing list