fix issue open-source-parsers#90

cdunn2001 · cdunn2001 · commit 7d82b1472674 · 2015-01-24T14:34:54.000-06:00
We are static-casting to U, so we really have no reason to use references. However, if this comes up again, try applying -ffloat-store to the target executable, per open-source-parsers#90
diff --git a/src/test_lib_json/CMakeLists.txt b/src/test_lib_json/CMakeLists.txt
@@ -11,6 +11,9 @@ ADD_EXECUTABLE( jsoncpp_test
 
 TARGET_LINK_LIBRARIES(jsoncpp_test jsoncpp_lib)
 
+# another way to solve issue #90
+#set_target_properties(jsoncpp_test PROPERTIES COMPILE_FLAGS -ffloat-store)
+
 # Run unit tests in post-build
 # (default cmake workflow hides away the test result into a file, resulting in poor dev workflow?!?)
 IF(JSONCPP_WITH_POST_BUILD_UNITTEST)
diff --git a/src/test_lib_json/jsontest.h b/src/test_lib_json/jsontest.h
@@ -178,8 +178,8 @@ class Runner {
 
 template <typename T, typename U>
 TestResult& checkEqual(TestResult& result,
-                       const T& expected,
-                       const U& actual,
+                       T expected,
+                       U actual,
                        const char* file,
                        unsigned int line,
                        const char* expr) {
