You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Many classes in ViSP have a standard buildFrom method, which serves as a delayed constructor/initializer.
In many classes, this method has the signature vpObject buildFrom(Args... args). However, buildFrom modifies the calling object and there is very little sense in returning a copy.
This method should either:
Return void: this method acts as a setter
Return a reference: Allows for method chaining e.g. x.buildFrom(args).compute();
Currently, we can perform method chaining, but this is misleading
Some of the affected classes are:
vpRzyxVector, vpRxyzVector, vpRzyzVector
vpThetaUVector
vpQuaternionVector
vpRotationMatrix
vpTranslationVector
vpPoseVector
vpVelocityTwistMatrix
vpForceTwistMatrix
The text was updated successfully, but these errors were encountered:
Many classes in ViSP have a standard buildFrom method, which serves as a delayed constructor/initializer.
In many classes, this method has the signature
vpObject buildFrom(Args... args)
. However, buildFrom modifies the calling object and there is very little sense in returning a copy.This method should either:
Currently, we can perform method chaining, but this is misleading
Some of the affected classes are:
vpRzyxVector
,vpRxyzVector
,vpRzyzVector
vpThetaUVector
vpQuaternionVector
vpRotationMatrix
vpTranslationVector
vpPoseVector
vpVelocityTwistMatrix
vpForceTwistMatrix
The text was updated successfully, but these errors were encountered: