laas / metapod

A template-based robot dynamics library
GNU Lesser General Public License v3.0
14 stars 10 forks source link

spatial: remove OperatorMul and cleanup operators #58

Closed sbarthelemy closed 11 years ago

sbarthelemy commented 11 years ago

OperatorMul does not seem to be needed. I'm not sure about why it was introduced in the first place.

The only point I see is that Inertia could declare the generic OperatorMul as a friend and hence let future specializations access its private members. However, all its private members are accessible through getters, so this does not seem necessary.

Maybe I missed the point, feel free to discuss.

I checked that this change has no performance impact.