Bugzilla – Bug 439
Quaternion's `setFromTwoVectors` should be either/both of a static function or/and a constructor of Quaternion
Last modified: 2012-03-26 18:53:25 UTC
setFromTwoVectors AFAIK is (almost always?) used á la mode de AngleAxis or Identity.
As such it seems that it should exist as either a constructor or static function.
good point to add a FromTwoVectors static function. For the ctor, I'm not fan because it would be quite confusing.
Created attachment 260 [details]
Patch to add FromTwoVectors static function
Static function which calls the non-static.
Copy pasted the unit test, and ran unit test.
thanks for the patch, I committed a variant: this function has to be part of Quaternion, not QuaternionBase.
date: 2012-03-26 18:30:04
summary: fix bug 439: add Quaternion::FromTwoVectors() static constructor
Thanks for the quick action! : )