loplugin:useuniqueptr in OfaAutocorrExceptPage

Change-Id: If59ebc6cd5f73534e3e6c7e3765133c84d839958
Reviewed-on: https://gerrit.libreoffice.org/56325
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

2 files changed, 12 insertions, 17 deletions

diff --git a/cui/source/inc/autocdlg.hxx b/cui/source/inc/autocdlg.hxx
index ecef0e81a0be..017909ca7433 100644
--- a/cui/source/inc/autocdlg.hxx
+++ b/cui/source/inc/autocdlg.hxx
@@ -254,8 +254,8 @@ private:
         std::set<OUString>      aFormatText;
         std::map<LanguageType, DoubleStringArray>
                                 aDoubleStringTable;
-        CollatorWrapper*        pCompareClass;
-        CharClass*              pCharClass;
+        std::unique_ptr<CollatorWrapper>  pCompareClass;
+        std::unique_ptr<CharClass>        pCharClass;
         LanguageType            eLang;
 
         bool bHasSelectionText;
@@ -320,7 +320,7 @@ private:
     VclPtr<CheckBox>       m_pAutoCapsCB;
 
     StringsTable    aStringsTable;
-    CollatorWrapper* pCompareClass;
+    std::unique_ptr<CollatorWrapper> pCompareClass;
     LanguageType    eLang;
 
     DECL_LINK(NewDelButtonHdl, Button*, void);
diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx
index 97444eaec472..16a92adb7d0b 100644
--- a/cui/source/tabpages/autocdlg.cxx
+++ b/cui/source/tabpages/autocdlg.cxx
@@ -880,9 +880,9 @@ OfaAutocorrReplacePage::OfaAutocorrReplacePage( vcl::Window* pParent,
     bSWriter = pMod == SfxModule::GetActiveModule();
 
     LanguageTag aLanguageTag( eLastDialogLanguage );
-    pCompareClass = new CollatorWrapper( comphelper::getProcessComponentContext() );
+    pCompareClass.reset( new CollatorWrapper( comphelper::getProcessComponentContext() ) );
     pCompareClass->loadDefaultCollator( aLanguageTag.getLocale(), 0 );
-    pCharClass = new CharClass( aLanguageTag );
+    pCharClass.reset( new CharClass( aLanguageTag ) );
 
     static long const aTabs[] = { 1, 61 };
     m_pReplaceTLB->SetTabs( SAL_N_ELEMENTS(aTabs), aTabs );
@@ -913,10 +913,8 @@ void OfaAutocorrReplacePage::dispose()
     aDoubleStringTable.clear();
     aChangesTable.clear();
 
-    delete pCompareClass;
-    pCompareClass = nullptr;
-    delete pCharClass;
-    pCharClass = nullptr;
+    pCompareClass.reset();
+    pCharClass.reset();
 
     m_pTextOnlyCB.clear();
     m_pShortED.clear();
@@ -1099,13 +1097,11 @@ void OfaAutocorrReplacePage::SetLanguage(LanguageType eSet)
     {
         RefillReplaceBox(false, eLang, eSet);
         eLastDialogLanguage = eSet;
-        delete pCompareClass;
-        delete pCharClass;
 
         LanguageTag aLanguageTag( eLastDialogLanguage );
-        pCompareClass = new CollatorWrapper( comphelper::getProcessComponentContext() );
+        pCompareClass.reset( new CollatorWrapper( comphelper::getProcessComponentContext() ) );
         pCompareClass->loadDefaultCollator( aLanguageTag.getLocale(), 0 );
-        pCharClass = new CharClass( aLanguageTag );
+        pCharClass.reset( new CharClass( aLanguageTag ) );
         ModifyHdl(*m_pShortED);
     }
 }
@@ -1399,7 +1395,7 @@ OfaAutocorrExceptPage::OfaAutocorrExceptPage(vcl::Window* pParent, const SfxItem
     get(m_pAutoCapsCB, "autodouble");
 
     css::lang::Locale aLcl( LanguageTag::convertToLocale(eLastDialogLanguage ));
-    pCompareClass = new CollatorWrapper( comphelper::getProcessComponentContext() );
+    pCompareClass.reset( new CollatorWrapper( comphelper::getProcessComponentContext() ) );
     pCompareClass->loadDefaultCollator( aLcl, 0 );
 
     m_pNewAbbrevPB->SetClickHdl(LINK(this, OfaAutocorrExceptPage, NewDelButtonHdl));
@@ -1424,7 +1420,7 @@ OfaAutocorrExceptPage::~OfaAutocorrExceptPage()
 void OfaAutocorrExceptPage::dispose()
 {
     aStringsTable.clear();
-    delete pCompareClass;
+    pCompareClass.reset();
     m_pAbbrevED.clear();
     m_pAbbrevLB.clear();
     m_pNewAbbrevPB.clear();
@@ -1570,8 +1566,7 @@ void OfaAutocorrExceptPage::SetLanguage(LanguageType eSet)
         // save old settings and fill anew
         RefillReplaceBoxes(false, eLang, eSet);
         eLastDialogLanguage = eSet;
-        delete pCompareClass;
-        pCompareClass = new CollatorWrapper( comphelper::getProcessComponentContext() );
+        pCompareClass.reset( new CollatorWrapper( comphelper::getProcessComponentContext() ) );
         pCompareClass->loadDefaultCollator( LanguageTag::convertToLocale( eLastDialogLanguage ), 0 );
         ModifyHdl(*m_pAbbrevED);
         ModifyHdl(*m_pDoubleCapsED);