Closed kshitijkshitij closed 9 years ago
Looks like Microsoft being late on C++11 support :-)
I'm sure Séverin will LOVE helping you fix that on windows/VS ;) As I told Mirko, maybe it would be wiser to use the big_refactoring branch if you're going to fight for the wiwin compilation. Did you run CMake with WITH_LSD on ? Or did you just shortcut cmake and make your own proj files ?
<context>
we have spend 15 min with @kshitijkshitij fighting with CMake on Windows, to eventually successfully generate a VS project that seems to be correct</context>
The actual issue is related to VS2012 not supporting initializer lists for vectors. The question then is: can we refrain to use them?
And use tons of push_back all over ? wel I guess we can...
What branch are you on ?
Maybe it's a small syntactic difference ? I remember one like that between gcc and clang. Have you tried variations like:
std::vector
Note that this bug does not appear with VS2013 + C++11. Should we mark VS2012 as not supported and close this issue?
Hum, tough one... I'm about as enthusiastic as you are to dumb down the code to support legacy proprietary software, but the poor windows people can't always upgrade that easily...
I guess it depends on the effort needed to support VS12. Would that be the only issue ? E.g. once we fix the brace initialisers, won't more c++11 pop up ? If not, it seems a good investment to fix these compilation errors, because the code changed quite a lot in the last 3 months and I think it went in favour of this issue (no more LSD, and some brace initialisations disappeared because of the use of Matx44). The big problem would rather be to find a volunteer equipped with VS12...
Okay, let's forget about non C++11 compilers... Welcome to 2014 ;)
Here I post the errors for each file;