Hi, I'm not in the team, but I can give you some suggestions:
#defines are not nice for constants. they can be used, but its not nice. const's are better. #defines are for macros
I also suggest defining the constants inside classes (or namespaces?) to avoid conficts. To avoid defining constant for each instance of class making it static is a good way to go.
If you are defining 'int' constants which you want to use in the definition of code (for example as size of the array) then const would not work, in that case I suggest to define it as in enumerate.
Example:
header:
class GearsFileDefinition {
static const int number_of_bones;
static const std::string init_file_name;
enum { filename_length = 34 } ;
static const char filename[ filename_length ];
} ;
cpp:
// initialization of static variables
const int GearsFileDefinition::number_of_bones = 4;
const std::string GearsFileDefinition::init_file_name = "gears.ini";
const char GearsFileDefinition::filename[ GearsFileDefinition::filename_length ] = { "Hello" } ;
usage inside the class:
for( i=0; i<number_of_bones; i++ ) {
...
}
usage outside of class:
for( i=0; i<GearsFileDefinition::number_of_bones; i++ ) {
...
}
Log( GearsFileDefinition::filename );
Hope that you understand the code, if you don't please ask.
gigaherz: naming convention depends on users. I saw few of conventions and each was hated by someone else