VS2013 compile error when you define the static const variable in class of head file

[tunabrain]

VS2013 compile error when you define the static const variable in class of head file

[tunabrain]

Hi,

I tested locally on VS2013 and VS2015, but did not encounter the same error. I also set up automated VS2013 builds recently to make sure, but the builds succeed. Is it possible that you modified the project, or do these errors also occur in a freshly cloned repository?

[WindyPaper]

Thanks for your reply. I do these errors in a freshly cloned repository. Maybe there are little different settings in my environment. I will test whether this error will encounter on VS2015.

[mmmovania]

Yeah I am also getting the same (symbol already defined linker errors) on the latest (master) branch in VS2013. I am compilimg using vs2013 ultimate on Windows 8.1 64-bit. The code compiles and builds without errors on VS2015.

On VS2013, the solution is to comment the constant declaration line inside the namespace tungsten in the .cpp file. For example, in BitManip.cpp file

#include "BitManip.hpp"

#include <cmath>

namespace Tungsten {

//CONSTEXPR uint32 BitManip::LogMantissaBits; //comment this line

std::unique_ptr<float[]> BitManip::_logLookup;

BitManip::Initializer BitManip::initializer;

Just a suggestion, would it be better to put the TILE_SIZE variable in the parent class (Integrator) rather than declaring and defining in each child class?