2015-03-15 18:47 GMT+01:00 Jean-Marc Lasgouttes:

> We do not have a use case yet, but you commit that minus the part that
> defines the set<> version.


I am not really sure it makes sense if we just need VectorFromString, since
the change replaces push_back with insert in the template (since Set does
not have push_back), and this operation is apparently a bit more expensive
(so I learned).


> Actually, it seems to me that you do not Need the String template
> parameter. Container::value_type should do the trick, IMO.
>

OK. This somewhat exceeds my current C++ expertise, so I'd rather let you
(or someone else) do that change.

Jürgen


>
> JMarc
>
>

Reply via email to