Andy Maloney and Jesse Beder
b650bc8287
Modernize: Use range-based for loops for readability ( #762 )
...
Also run clang-format on these files as requested
2019-10-05 14:20:17 -05:00
Andy Maloney and Jesse Beder
848091124d
Replace node's YAML_CPP_OPERATOR_BOOL macro with an explicit bool operator ( #755 )
2019-10-02 14:14:07 -05:00
Andy Maloney and Jesse Beder
6e87b37034
Modernize: Use "using" instead of "typedef" ( #754 )
2019-10-02 11:14:49 -05:00
Andy Maloney and Jesse Beder
0fddd1e5bd
Modernization: Use "override" when overriding base class methods ( #753 )
2019-10-02 09:21:17 -05:00
Andy Maloney and Jesse Beder
99d95d8edc
Pass std::string by const ref ( #749 )
2019-10-02 08:39:41 -05:00
Andy Maloney and Jesse Beder
a6ed66abca
Modernize: Use "default" for destructors and copy constructors ( #751 )
2019-10-02 08:38:49 -05:00
Andy Maloney and Jesse Beder
e6b3a92e67
Use cassert instead of old assert.h ( #750 )
2019-10-02 08:37:38 -05:00
Andy Maloney and Jesse Beder
47d32aadf0
CMake: Remove unused option YAML_APPLE_UNIVERSAL_BIN ( #744 )
2019-09-27 12:26:35 -05:00
Andy Maloney and Jesse Beder
968e0c1f02
Fix shared lib build with new YAML_BUILD_SHARED_LIBS option ( #737 )
2019-09-14 21:21:35 -04:00
Andy Maloney and Jesse Beder
b218787b98
Remove extraneous conversion from string -> c_str -> string ( #739 )
2019-09-14 20:24:17 -04:00
Andy Maloney and Jesse Beder
db0bda7087
Static analysis fix: replace 0 and NULL with nullptr ( #738 )
2019-09-14 20:23:22 -04:00
Andy Maloney and Jesse Beder
6cdf363625
CMake: Prefix options with "YAML" and hide platform-specific options ( #734 )
...
* CMake: Prefix options with "YAML" and hide platform-specific options
When including yaml-cpp as a subproject, some option names can conflict with other projects.
(1) Make sure the yaml-cpp options are prefixed with YAML
(2) Hide platform-specific options when possible to avoid cluttering the cmake option list
* Update docs for change from BUILD_SHARED_LIBS to YAML_BUILD_SHARED_LIBS
2019-09-10 10:00:07 -07:00
Andy Maloney and Jesse Beder
d638508d33
Set C++ standard options in CMake per-target instead of globally ( #735 )
...
Setting CMAKE_CXX_STANDARD and CMAKE_CXX_STANDARD_REQUIRED directly is problematic when including yaml-cpp as a subproject.
The proper way is to set these per-target.
2019-09-10 07:51:13 -07:00
Andy Maloney and Jesse Beder
90350662c9
Use VERSION on the CMake project ( #733 )
...
This sets the other variables:
https://cmake.org/cmake/help/latest/command/project.html
2019-09-10 07:50:10 -07:00
Andy Maloney and Jesse Beder
ee99c4151c
Fix a warning from -Wshadow ( #627 )
...
Variable "it" was shadowed
2018-09-05 23:16:02 -05:00
Andy MaloneyandJesse Beder
