[PATCH] cppcheck: fix uselessAssignmentPtrArg

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sun May 12 09:10:02 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3873

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/73/3873/1

cppcheck: fix uselessAssignmentPtrArg

Change-Id: I260bb1207cebb8cefb52fe1db3f51b63239b25a7
---
M sc/source/core/tool/consoli.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sc/source/core/tool/consoli.cxx b/sc/source/core/tool/consoli.cxx
index 286261e..5f05383 100644
--- a/sc/source/core/tool/consoli.cxx
+++ b/sc/source/core/tool/consoli.cxx
@@ -420,7 +420,7 @@
 }
 
 static double lcl_CalcData( ScSubTotalFunc eFunc,
-                        double fCount, double fSum, double fSumSqr)
+                        double& fCount, double fSum, double fSumSqr)
 {
     if (fCount < 0.0)
         return 0.0;

-- 
To view, visit https://gerrit.libreoffice.org/3873
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I260bb1207cebb8cefb52fe1db3f51b63239b25a7
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list