[Libreoffice-commits] core.git: scaddins/source

Markus Mohrhard markus.mohrhard at googlemail.com
Fri Aug 5 18:28:29 UTC 2016


 scaddins/source/analysis/analysis.cxx |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

New commits:
commit dc72abc793baa57199a5674c066df5ec024d1771
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Thu Aug 4 15:53:22 2016 +0200

    no need to check for nullptr before deleting
    
    Change-Id: I5512ad52c9a0f943b29656e16cceeb08ce164abf
    Reviewed-on: https://gerrit.libreoffice.org/27888
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/scaddins/source/analysis/analysis.cxx b/scaddins/source/analysis/analysis.cxx
index a8d9be1..eb53f0e 100644
--- a/scaddins/source/analysis/analysis.cxx
+++ b/scaddins/source/analysis/analysis.cxx
@@ -138,11 +138,8 @@ void AnalysisAddIn::InitData()
         pFD = nullptr;
     }
 
-    if( pDefLocales )
-    {
-        delete pDefLocales;
-        pDefLocales = nullptr;
-    }
+    delete pDefLocales;
+    pDefLocales = nullptr;
 }
 
 AnalysisAddIn::AnalysisAddIn( const uno::Reference< uno::XComponentContext >& xContext ) :


More information about the Libreoffice-commits mailing list