[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [IMP-dev] [IMP-commits] r1297 - trunk/modules/algebra/include



Daniel Russel wrote:
> It also means that when someone creates a default Transformation3D, you
> don't know if they want the identity transform or just will initialize
> it later. And it's not symmetric with the Vector3D and the Rotation3D.

I agree - I would prefer default constructed objects to have invalid or
at least undefined state. If you want the identity, you should
explicitly initialize it that way - makes your code easier to follow.

	Ben
-- 
                      http://salilab.org/~ben/
"It is a capital mistake to theorize before one has data."
	- Sir Arthur Conan Doyle