search

favreau / bullet

Automatically exported from code.google.com/p/bullet
0 stars 0 forks source link

579 warnings #443

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Build something using Bullet with -Wall -Weffc++

There are 579 warnings (at least) about members not being initialized in the 
initialization list.

For example:
stuct MyClass
{
int num;
MyClass(): num(5) {}
};

You really should always initialize the variables of classes, otherwise you 
risk using bad memory/values. Also, variables should be initialized in the 
initializer list, not set in the body due to speed (IIRC).

Thanks,
Jim

Original issue reported on code.google.com by johaga...@gmail.com on 27 Oct 2010 at 2:04

GoogleCodeExporter commented 9 years ago 

It has not been a coding guideline to require all members to be initialized in 
the initializer list. Unless someone provides a clean patch (exclusively 
dealing with this initializer list), we don't allocate resources to fix all 
-Weffc++ warnings for Bullet 2.x.

We'll consider supporting this for Bullet 3.x though.
Thanks for the feedback!

Original comment by erwin.coumans on 27 Oct 2010 at 11:32