On 02/13/2008 10:45 AM, Hans de Goede wrote:
Attached is a patch which fixes this by un-inlining the Collision
constructor. Actually it seems that the best way to fix this is to
un-inline all Collision members which use the template and move the
template entirely to Collision.cpp
And since you are using C++, don't hesitate to take advantage of
initialization list, either. E.g.:
+ CollisionSystem::CollisionSystem()
+ : m_pGrid(NULL), m_bDebugFlag(false)
+ {
+ }
Since you're using templates, it may be a win in this specific case (I
haven't looked too deeply), but nonetheless it is a good habit to get
into as it can have performance benefits. Just make sure you keep the
list in the same order as declared in the class.
--
fedora-devel-list mailing list
fedora-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-devel-list
