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

Markus Mohrhard markus.mohrhard at googlemail.com
Sun Mar 20 22:13:14 UTC 2016


 sc/source/core/tool/interpr5.cxx |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 143ab8ded1ecfdb32f634a50ede970b0994bdf84
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Sun Mar 20 22:03:52 2016 +0100

    no need to declare the variable outside of the for loop
    
    Change-Id: I1289d7e9b5f3459882582b494006a69075a0b860
    Reviewed-on: https://gerrit.libreoffice.org/23382
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/sc/source/core/tool/interpr5.cxx b/sc/source/core/tool/interpr5.cxx
index 479b9ca..22a8375 100644
--- a/sc/source/core/tool/interpr5.cxx
+++ b/sc/source/core/tool/interpr5.cxx
@@ -1163,7 +1163,6 @@ ScMatrixRef ScInterpreter::MatConcat(const ScMatrixRef& pMat1, const ScMatrixRef
 {
     SCSIZE nC1, nC2, nMinC;
     SCSIZE nR1, nR2, nMinR;
-    SCSIZE i, j;
     pMat1->GetDimensions(nC1, nR1);
     pMat2->GetDimensions(nC2, nR2);
     nMinC = lcl_GetMinExtent( nC1, nC2);
@@ -1171,9 +1170,9 @@ ScMatrixRef ScInterpreter::MatConcat(const ScMatrixRef& pMat1, const ScMatrixRef
     ScMatrixRef xResMat = GetNewMat(nMinC, nMinR);
     if (xResMat)
     {
-        for (i = 0; i < nMinC; i++)
+        for (SCSIZE i = 0; i < nMinC; i++)
         {
-            for (j = 0; j < nMinR; j++)
+            for (SCSIZE j = 0; j < nMinR; j++)
             {
                 sal_uInt16 nErr = pMat1->GetErrorIfNotString( i, j);
                 if (!nErr)


More information about the Libreoffice-commits mailing list