tree a25f2c6cbca2cede50ecdafa37207f4d57661a00
parent d4742c2b4550d4ee516f065c87315c756ac2430f
author Jordan Bayles <bayles.jordan@gmail.com> 1773639497 -0700
committer GitHub <noreply@github.com> 1773639497 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJpt5dJCRC1aQ7uu5UhlAAAVS8QAA5Wvmof2kclZVvTvDzq8znW
 7YwEnXO5xbO0s9X8fE4z5Mke8zif3x5l6RLJlbKgee32ozXyE8bn4hn1KfxRv2Xk
 ZJuo4NFmNKmFd0+Z6L9bWzutcugshT6979cNCyVDugOCstKuZ6/sBeJXpAErpZm5
 6/LSN+W7U4Ygsc+UZGc7SuBzRwXeWiXavkjcEgY9EPNzJLOenOHKIfJ1kL+qEsQw
 uJSY0hSzLiKC3gMYq+ncm+BietYS/WNsn75/oDJKJOBJW8PV/A5fqP8oLMMIG5qi
 QAzvVKNtCTVQXGEJZWnGn1Hev/ISkjzpBMg03/rAdlHvBxo/gtZM1c3ei9Q3gyU9
 0p3vNH0C/alkPY7m4a1C/wPj7530EDDNW+ZA5bpMjhDas98UizQmavpqVUAqaR9F
 1RtuHeb817E+PmTIg9p2pYz7lOlFRjzQ0qcUZvfkuawSepekOoHWKyOv1XGQylvm
 yFeHG5Gt6GvVKBNS1brLE12+hlQ7UgPWWYGVkjci6b/tRIXhnJp2N/ZexFPIBEjY
 bW8ssLXBIG30PW8adVxYSj/DnhLiHkw1nYG5ofWBv8tN3r6dY4YOAdQ+DL5rGagz
 s9aj0intvU4dxtZ79XJDPCM1CEjkhj0G0PuX0ZPoys+vny8pstucNbtXcprSUuFn
 FOCPZCju+vDXel/ii/uI
 =M5Va
 -----END PGP SIGNATURE-----
 

Fix string_view ABI mismatch between library and consumers (#1661)

The library was forced to build with CMAKE_CXX_STANDARD 11, so
JSONCPP_HAS_STRING_VIEW was never defined at compile time. Consumers
building with C++17 would see the string_view APIs in the header but
fail to link them.

Fix:
- Remove the global CMAKE_CXX_STANDARD 11 override; the existing
  target_compile_features(cxx_std_11) already enforces the minimum.
- Detect string_view support at configure time with
  check_cxx_source_compiles and export JSONCPP_HAS_STRING_VIEW as a
  PUBLIC compile definition on all library targets, so consumers
  always see the same value the library was built with.
- Guard the __cplusplus fallback in value.h so it does not override
  the CMake-set define.

Fixes #1595