helper containers does not have c++11 operators about sofa HOT 5 CLOSED

helper/map.h is implemented that way, and it works.
We should do the same for set, list, pair, deque, stable_vector.

And we should add the new operators to helper::vector and helper::SVector.

from sofa.

started in https://github.com/anatoscope/sofa/tree/containers

from sofa.

Wouldn't it make sense to alias the std container to sofa::helper ?

So that the general guideline could be to always use sofa::helper::* and not "if your are using a list then use std::list, if it is a vector then use a sofa::helper::vector, ...".

More generally

• it would make the source code more consistent
• it would ease newcommer's life to get into sofa.

from sofa.

Also we could just define the stream operators << >> directly on std::vector (with the actual helper::vector implementations). So std::vector could be used in Data and helper::vector would be useful only for advanced users (like cuda-vector, stuff that is not necessary for regular Data).

from sofa.

There is a PR that resulted from this discussion and it was merged.
I close the Issue, do not hesitate to reopen in case it is too early.

from sofa.