Massrebuilds for GCC 4.3 coming soon to a buildsystem near you!
Christopher Aillon
caillon at redhat.com
Wed Feb 13 16:21:33 UTC 2008
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.
More information about the devel
mailing list