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

Dennis Francis dennis.francis at collabora.co.uk
Tue Nov 21 07:01:11 UTC 2017


Rebased ref, commits from common ancestor:
commit 59b501be351d03e3bc0d6283dfb8d3341c0e224d
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());
diff --git a/sc/source/core/tool/formulagroup.cxx b/sc/source/core/tool/formulagroup.cxx
index 3acc38d68051..f6565e732086 100644
--- a/sc/source/core/tool/formulagroup.cxx
+++ b/sc/source/core/tool/formulagroup.cxx
@@ -377,7 +377,7 @@ bool FormulaGroupInterpreterSoftware::interpret(ScDocument& rDoc, const ScAddres
         }
         SCROW nRemaining = nLen - nBatchSize * nThreadCount;
 
-        SAL_INFO("sc.swipret.threaded", "Running " << nThreadCount << " threads");
+        SAL_INFO("sc.threaded", "Running " << nThreadCount << " threads");
 
         SCROW nLeft = nLen;
         SCROW nStart = 0;
@@ -393,9 +393,9 @@ bool FormulaGroupInterpreterSoftware::interpret(ScDocument& rDoc, const ScAddres
             nLeft -= nCount;
             nStart = nLast + 1;
         }
-        SAL_INFO("sc.swipret.threaded", "Joining threads");
+        SAL_INFO("sc.threaded", "Joining threads");
         rThreadPool.waitUntilDone(aTag);
-        SAL_INFO("sc.swipret.threaded", "Done");
+        SAL_INFO("sc.threaded", "Done");
     }
     else
     {


More information about the Libreoffice-commits mailing list