[PATCH] coverity#983561: Arguments in wrong order

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sat Mar 30 11:04:59 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3126

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/26/3126/1

coverity#983561: Arguments in wrong order

Change-Id: I3035457e93a659fe09cfe383735328f2854e5871
---
M basegfx/source/polygon/b3dpolypolygontools.cxx
1 file changed, 4 insertions(+), 3 deletions(-)



diff --git a/basegfx/source/polygon/b3dpolypolygontools.cxx b/basegfx/source/polygon/b3dpolypolygontools.cxx
index dd7fa41..2f498be 100644
--- a/basegfx/source/polygon/b3dpolypolygontools.cxx
+++ b/basegfx/source/polygon/b3dpolypolygontools.cxx
@@ -222,9 +222,10 @@
             return aRetval;
         }
 
-        // helper for getting the 3D Point from given cartesian coordiantes. fVer is defined from
-        // [F_PI2 .. -F_PI2], fHor from [0.0 .. F_2PI]
-        inline B3DPoint getPointFromCartesian(double fVer, double fHor)
+        // helper for getting the 3D Point from given cartesian coordinates.
+        // fHor from [0.0 .. F_2PI]
+        // fVer is defined from [F_PI2 .. -F_PI2]
+        inline B3DPoint getPointFromCartesian(double fHor, double fVer)
         {
             const double fCosHor(cos(fHor));
             return B3DPoint(fCosHor * cos(fVer), sin(fHor), fCosHor * -sin(fVer));

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

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



More information about the LibreOffice mailing list