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
    https://github.com/open-source-parsers/jsoncpp/issues/90
This commit is contained in:
Christopher Dunn 2015-01-24 14:34:54 -06:00
parent 2bc6137ada
commit 7d82b14726
2 changed files with 5 additions and 2 deletions

View File

@ -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)

View File

@ -178,8 +178,8 @@ private:
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) {