Enable c++11 support?

Issue #43 resolved
dsmic created an issue

I am not sure if it would be a good idea, but it would make initialization of the two vectors for circular pattern possible. It could reduce start up time for iPad e.g.

Pattern::CircularDictionary *circdict; 
std::map<Pattern::Circular,unsigned int> *circlevels;
std::map<unsigned int,std::string> *circstrings;
std::map<unsigned int,float> *circgammas;

//test c++11 Enable with -std=c++11

std::vector<int> test={1,2,3,4,5};

Comments (5)

  1. Francois van Niekerk repo owner

    The reason I have avoid upgrading to C++11 is because I don't have admin privileges for some of the machines that I test Oakfoam on and they are running older version of gcc without the support.

    Does this actually make a difference to loading time? If tests show that it actually makes a difference, I will revisit C++11.

    Personally I prefer to have the large files stored external to the binary. For one, it wouldn't help very much when you are testing new weights, because re-compiling with the prospective weights seems will cause many problems.

  2. dsmic reporter

    would be ok to me, but I have no idea how to write / read std::map. I will let you know,

    I think it might even be possible to use the file for compiling in the gammas only, if c++11 is availible.

    Detlef

  3. dsmic reporter

    hmm compiling it in does not seem to be an option, as gcc needs hours to do this for this kind of data structures, so if you see a way to load binary I would love it:)

    Detlef

  4. Francois van Niekerk repo owner

    Ok, it seems like this is resolved: enabling C++11 support will not fix the problem.

    A possible solution might be to generate a binary format, but this will probably be more work than it is worth.

  5. Log in to comment