[Libreoffice-commits] core.git: Branch 'feature/calc-parallel' - sc/source

Dennis Francis dennis.francis at collabora.co.uk
Tue Nov 21 06:41:50 UTC 2017


 sc/source/core/data/formulacell.cxx |    8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

New commits:
commit b860af31c67ff329afc9c4437b96bac9d04b9228
Author: Dennis Francis <dennis.francis at collabora.co.uk>
Date:   Tue Nov 21 12:11:01 2017 +0530

    fix some problems pointed out by loplugin
    
    Change-Id: I864384942621355acd263f1cf09f21610459bee2

diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx
index f4fb466f836b..329f1afa5034 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -1738,15 +1738,11 @@ void ScFormulaCell::Interpret()
 namespace {
 class StackCleaner
 {
-    ScInterpreter*  pInt;
+    std::unique_ptr<ScInterpreter>  pInt;
     public:
     StackCleaner( ScInterpreter* pI )
         : pInt(pI)
         {}
-    ~StackCleaner()
-    {
-        delete pInt;
-    }
 };
 }
 
@@ -4291,7 +4287,7 @@ struct ScDependantsCalculator
             }
         }
 
-        for (auto i = 0u; i < aRangeList.size(); ++i)
+        for (size_t i = 0; i < aRangeList.size(); ++i)
         {
             const ScRange* pRange = aRangeList[i];
             assert(pRange->aStart.Tab() == pRange->aEnd.Tab());


More information about the Libreoffice-commits mailing list