[PATCH] coverity#983561: Arguments in wrong order

Thomas Arnhold thomas-libo at arnhold.org
Sun Mar 31 06:21:10 PDT 2013


Hi Regina,

you're right, the other methods are using the order (fVer, fHor), like 
createUnitSpherePolyPolygon() does. So the definition is correct:

inline B3DPoint getPointFromCartesian(double fVer, double fHor)

But on the call side it's used like:

aNew.append(getPointFromCartesian(fHor, fVer));

So here got something mixed up.

Thomas


More information about the LibreOffice mailing list