ToAxisAngle, ToAngleAxis.

Minor issues with the Ogre API that can be trivial to fix
User avatar
mkultra333
Gold Sponsor
Gold Sponsor
Posts: 1894
Joined: Sun Mar 08, 2009 5:25 am
x 116

ToAxisAngle, ToAngleAxis.

Post by mkultra333 »

I don't know if this serves a purpose or not, and I don't use either, but while looking in the API I noticed that Ogre has

Quaternion::ToAngleAxis( Radian & rfAngle, Vector3 & rkAxis ) const

and

Matrix3::ToAxisAngle ( Vector3 & rkAxis, Radian & rfAngle ) const

Unless there's a logic to this, perhaps it should either be ToAngleAxis or ToAxisAngle for both.
"In theory there is no difference between practice and theory. In practice, there is." - Psychology Textbook.
User avatar
masterfalcon
OGRE Retired Team Member
OGRE Retired Team Member
Posts: 4270
Joined: Sun Feb 25, 2007 4:56 am
Location: Bloomington, MN
x 126

Re: ToAxisAngle, ToAngleAxis.

Post by masterfalcon »

Please file a bug marked Papercut for this and the other one that you listed the other day.
User avatar
mkultra333
Gold Sponsor
Gold Sponsor
Posts: 1894
Joined: Sun Mar 08, 2009 5:25 am
x 116

Re: ToAxisAngle, ToAngleAxis.

Post by mkultra333 »

Filed a bug report for this one, but I haven't posted any other papercuts recently so you might have me confused with someone else. Jabberwocky, perhaps?
"In theory there is no difference between practice and theory. In practice, there is." - Psychology Textbook.
User avatar
masterfalcon
OGRE Retired Team Member
OGRE Retired Team Member
Posts: 4270
Joined: Sun Feb 25, 2007 4:56 am
Location: Bloomington, MN
x 126

Re: ToAxisAngle, ToAngleAxis.

Post by masterfalcon »

Oh yeah, you're right.
User avatar
Jabberwocky
OGRE Moderator
OGRE Moderator
Posts: 2819
Joined: Mon Mar 05, 2007 11:17 pm
Location: Canada
x 218

Re: ToAxisAngle, ToAngleAxis.

Post by Jabberwocky »

[papercut] bugs submitted for my posts.
Image