Upgrade jsoncpp (#773)

Fixes #772 

Upgrade version of jsoncpp to 1.9.5 to include fix for deprecated sprintf usage.

Upgrading introduced a new unneeded-internal-declaration warning which is set as a warning instead of error.

Signed-off-by: The MathWorks, Inc. <jdicleme@mathworks.com>

Signed-off-by: The MathWorks, Inc. <jdicleme@mathworks.com>
Co-authored-by: Jeff DiClemente <jdicleme@mathworks.com>

compendium/tools/SCRCodeGen/CMakeLists.txt

@@ -20,6 +20,10 @@ include_directories(../../../third_party
 set(US_SCRCODEGEN_EXECUTABLE_TARGET SCRCodeGen)
 set(US_SCRCODEGEN_EXECUTABLE_OUTPUT_NAME ${US_SCRCODEGEN_EXECUTABLE_TARGET}${US_GLOBAL_VERSION_SUFFIX})
 
+if(US_COMPILER_CLANG OR US_COMPILER_APPLE_CLANG)
+  add_compile_options(-Wno-error=unneeded-internal-declaration)
+endif()
+
 add_executable(${US_SCRCODEGEN_EXECUTABLE_TARGET} ${_srcs} ${_private_headers})
 if (US_BUILD_TESTING)
   target_compile_definitions(${US_SCRCODEGEN_EXECUTABLE_TARGET} PRIVATE USING_GTEST)

framework/src/CMakeLists.txt

@@ -57,7 +57,6 @@ set(_srcs
   bundle/BundleVersion.cpp
   bundle/Constants.cpp
   bundle/CoreBundleContext.cpp
-  ../../third_party/jsoncpp.cpp
   ../../third_party/miniz.c
 )
 

framework/test/gtest/CMakeLists.txt

@@ -17,6 +17,10 @@ configure_file("${CMAKE_CURRENT_SOURCE_DIR}/TestingConfig.h.in" "${PROJECT_BINAR
 
 set(us_gtest_test_exe_name usFrameworkTests)
 
+if(US_COMPILER_CLANG OR US_COMPILER_APPLE_CLANG)
+  add_compile_options(-Wno-error=unneeded-internal-declaration)
+endif()
+
 include_directories(
   ${GTEST_INCLUDE_DIRS}
   ${GMOCK_INCLUDE_DIRS}

third_party/README

@@ -29,11 +29,8 @@ MIT License or Public Domain
 
 https://github.com/open-source-parsers/jsoncpp
 (formerly http://jsoncpp.sourceforge.net)
-0.10.6
+1.9.5
 
-Patches
-
- * Suppress -Wimplicit-fallthrough warning in GCC 7
 
 miniz
 -----