[PATCH] coverity#983561: Arguments in wrong order

[julien2412]

Hi Regina,

I knew I was right to submit this patch for review and not push it directly
:-)
In fact, I just picked up a coverity report and took a look to Opengrok (see
http://opengrok.libreoffice.org/search?q=getPointFromCartesian&project=core&defs=&refs=&path=&hist=).
Ovbiously, I know nothing about 3D coordinate system, now could argument
names be changed a little so it wouldn't confuse Coverity scan (and also me
:-) ) ? (perhaps include your explanation in source code or a link to
Wikipedia or something). I had searched on Wikipedia but didn't find/didn't
know how to search about this.

Thank you for your useful feedback Regina!

Kind regards too

Julien



--
View this message in context: http://nabble.documentfoundation.org/PATCH-coverity-983561-Arguments-in-wrong-order-tp4047117p4047164.html
Sent from the Dev mailing list archive at Nabble.com.