From: Timothy S. <ti...@st...> - 2004-08-04 15:46:56
|
On Tue, 2004-08-03 at 09:36, Matthias Baas wrote: > As a second step I'd like to split the Space class in its individual=20 > classes as there are already three of them in ODE: SimpleSpace,=20 > HashSpace and QuadTreeSpace. However, before doing so I thought I'd=20 > better ask what you think of it. Are you in favor of this change or do=20 > you think we should stick to one Space class and pass the type as=20 > argument (as it is the case right now)? I'm in favour of splitting them into separate classes. > If we'll have individual classes, what class names would you prefer?=20 SimpleSpace, HashSpace and QuadTreeSpace are fine. > Should "Space" be the abstract base class or an alias for the=20 > SimpleSpace class to remain at least a bit of compatibility to the=20 > previous version? Or should "Space" be a factory function that still=20 > works like before and returns the appropriate space class..... I agree that having Space as a factory function is a good idea. Maybe "SpaceBase" could be the abstract base class for the spaces? --=20 Timothy Stranex <ti...@st...> |