You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
move std::optional and std::variant conversion support, which are currently in pybind11/stl.h, to a new file (say pybind11/utility.h since they are in the STL utility library in C++17).
include pybind11/utility.h from pybind11/stl.h so that we maintain full backward compatibility.
update the docs Overview page to mention the new header.
If you like the idea, I'm happy to prepare a pull request. Please let me know. #
The text was updated successfully, but these errors were encountered:
funchal
changed the title
Separate optional and variant support from other containers stl.h
Separate optional and variant support from other STL containers
Dec 7, 2017
Quick reminder, there is still interest in resolving this issue. stl.h has too much stuff in it some of it I want and some I don't. I currently have to workaround this would be great to get some solution in place.
I'd like to propose that we:
std::optional
andstd::variant
conversion support, which are currently inpybind11/stl.h
, to a new file (saypybind11/utility.h
since they are in the STL utility library in C++17).pybind11/utility.h
frompybind11/stl.h
so that we maintain full backward compatibility.My motivation for this proposal is that the conversion for other containers (set/vector/map) in
pybind11/stl.h
comes with a major downside, documented in http://pybind11.readthedocs.io/en/stable/advanced/cast/stl.html#automatic-conversion, and I believe that optional/variant can be useful in their own right.If you like the idea, I'm happy to prepare a pull request. Please let me know. #
The text was updated successfully, but these errors were encountered: