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

Caolán McNamara (via logerrit) logerrit at kemper.freedesktop.org
Sat Jul 20 18:49:11 UTC 2019


 sc/source/core/data/column4.cxx              |    4 ++--
 sc/source/core/data/table7.cxx               |   10 +++++-----
 vcl/source/bitmap/BitmapScaleSuperFilter.cxx |    6 +++---
 3 files changed, 10 insertions(+), 10 deletions(-)

New commits:
commit 90011180cb21186dbd58b8be6d7ffff23416541e
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sat Jul 20 15:45:21 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sat Jul 20 20:48:29 2019 +0200

    cid#1448415 Pointer to local outside scope
    
    Change-Id: I35f20fa37a19ff921916cd0e8bb7bafbc66ec8b5
    Reviewed-on: https://gerrit.libreoffice.org/76021
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sc/source/core/data/column4.cxx b/sc/source/core/data/column4.cxx
index 4485de036c8d..ac16dd2aa939 100644
--- a/sc/source/core/data/column4.cxx
+++ b/sc/source/core/data/column4.cxx
@@ -290,14 +290,14 @@ void ScColumn::CopyOneCellFromClip( sc::CopyFromClipContext& rCxt, SCROW nRow1,
         // Duplicate the cell note over the whole pasted range.
 
         ScDocument* pClipDoc = rCxt.getClipDoc();
-        const ScAddress& rSrcPos = pClipDoc->GetClipParam().getWholeRange().aStart;
+        const ScAddress aSrcPos = pClipDoc->GetClipParam().getWholeRange().aStart;
         std::vector<ScPostIt*> aNotes;
         ScAddress aDestPos(nCol, nRow1, nTab);
         aNotes.reserve(nDestSize);
         for (size_t i = 0; i < nDestSize; ++i)
         {
             bool bCloneCaption = (nFlags & InsertDeleteFlags::NOCAPTIONS) == InsertDeleteFlags::NONE;
-            aNotes.push_back(pNote->Clone(rSrcPos, *pDocument, aDestPos, bCloneCaption).release());
+            aNotes.push_back(pNote->Clone(aSrcPos, *pDocument, aDestPos, bCloneCaption).release());
             aDestPos.IncRow();
         }
 
diff --git a/sc/source/core/data/table7.cxx b/sc/source/core/data/table7.cxx
index 48576f0d6018..203b1e417dbb 100644
--- a/sc/source/core/data/table7.cxx
+++ b/sc/source/core/data/table7.cxx
@@ -162,14 +162,14 @@ void ScTable::CopyOneCellFromClip(
         OSL_ENSURE(pDrawLayer, "No drawing layer");
         if (pDrawLayer)
         {
-            const ScAddress& rSrcStartPos
+            const ScAddress aSrcStartPos
                 = rCxt.getClipDoc()->GetClipParam().getWholeRange().aStart;
-            const ScAddress& rSrcEndPos = rCxt.getClipDoc()->GetClipParam().getWholeRange().aEnd;
+            const ScAddress aSrcEndPos = rCxt.getClipDoc()->GetClipParam().getWholeRange().aEnd;
             tools::Rectangle aSourceRect = rCxt.getClipDoc()->GetMMRect(
-                rSrcStartPos.Col(), rSrcStartPos.Row(), rSrcEndPos.Col(), rSrcEndPos.Row(),
-                rSrcStartPos.Tab());
+                aSrcStartPos.Col(), aSrcStartPos.Row(), aSrcEndPos.Col(), aSrcEndPos.Row(),
+                aSrcStartPos.Tab());
             tools::Rectangle aDestRect = GetDoc().GetMMRect(nCol1, nRow1, nCol2, nRow2, nTab);
-            pDrawLayer->CopyFromClip(rCxt.getClipDoc()->mpDrawLayer.get(), rSrcStartPos.Tab(),
+            pDrawLayer->CopyFromClip(rCxt.getClipDoc()->mpDrawLayer.get(), aSrcStartPos.Tab(),
                                      aSourceRect, ScAddress(nCol1, nRow1, nTab), aDestRect);
         }
     }
commit 1f3c1b4ca12400348f30f30c2fb7ddfe2c71b2f7
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sat Jul 20 15:27:38 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sat Jul 20 20:48:15 2019 +0200

    cid#1448542 Unintended sign extension
    
    Change-Id: I5ef8661a1c8e28537c96cb899d124012938f4b1f
    Reviewed-on: https://gerrit.libreoffice.org/76017
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/vcl/source/bitmap/BitmapScaleSuperFilter.cxx b/vcl/source/bitmap/BitmapScaleSuperFilter.cxx
index 7f7d5bffdd3c..acb3ad04f2a3 100644
--- a/vcl/source/bitmap/BitmapScaleSuperFilter.cxx
+++ b/vcl/source/bitmap/BitmapScaleSuperFilter.cxx
@@ -790,9 +790,9 @@ void scaleDown24bit(ScaleContext &rCtx, long nStartY, long nEndY)
                                 1 : (rCtx.maMapIX[nRight] - rCtx.maMapIX[nLeft]);
             }
 
-            long nSum1 = 0;
-            long nSum2 = 0;
-            long nSum3 = 0;
+            int nSum1 = 0;
+            int nSum2 = 0;
+            int nSum3 = 0;
             BilinearWeightType nTotalWeightY = 0;
 
             for (long i = 0; i<= nLineRange; i++)


More information about the Libreoffice-commits mailing list