GNUInstallDirs provided may be absolute paths, in which case appending
to the install prefix is not correct. We can instead use the provided
CMAKE_INSTALL_FULL_* variables, which are precomputed absolute paths.
https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html
* Export YAML::detail::node::m_amount
The internal header node/detail/node.h is included by public headers;
YAML::detail::node is implemented in the header itself, and thus it gets
inlined... except for its static m_amount class member, which is
instantiated in the library only. Right now all the symbols of yaml-cpp
are exported (nothing is hidden), so the linker will find node::m_amount
in the yaml-cpp library.
As solution/workaround, explicitly export YAML::detail::node::m_amount.
* CMake: use GenerateExportHeader
Make use of the GenerateExportHeader CMake module to generate the dll.h
header with export macros.
While the produced dll.h is different, the result should be the same,
i.e. nothing changes for yaml-cpp or its users.
* CMake: hide all the symbols by default
Hide all the symbols that are not explicitly exported with YAML_CPP_API.
This way the ABI will be way smaller, and only actually exposing the
public classes/functions.
- Don't eagerly convert key to std::string
- Make const char* keys streamable when exception is thrown
- Don't create a temporary string when comparing a const char* key
Jesse BederandGitHub
Pavel KarelinandGitHub
Ryan BurnsandGitHub
PhilipDeeganandGitHub
Robert Sebastian HerlimandGitHub
Michał JaniszewskiandGitHub
João Marcos BezerraandGitHub
Pino ToscanoandGitHub
jwindgassenandGitHub
Snow PettersenandGitHub
mjvankampenandGitHub
jbeachandGitHub
JBPenningtonandGitHub
theamarinandGitHub
t.tandGitHub
ChenandGitHub
Kajetan ŚwierkandGitHub