fix #1345: don't remove big endian test for other platforms 1354/head
authorMehdi Sabwat <mehdi@videolabs.io>
Tue, 1 Jun 2021 20:39:27 +0000 (22:39 +0200)
committerMehdi Sabwat <mehdi@videolabs.io>
Tue, 1 Jun 2021 20:39:27 +0000 (22:39 +0200)
thanks to @ePirat, it turns out that !EMSCRIPTEN does not evaluate to 1 on other platforms, so the the #1345 actually disabled the test for all platforms.

CMakeLists.txt

index 28fc5fe1cde2927e9e94c9c48b0e4831670c89b9..d343905b07466091e7a2b69f23c9f798a39564e5 100644 (file)
@@ -162,7 +162,7 @@ endif()
 
 #-----------------------------------------------------------------------------
 # Big endian test:
-if (!EMSCRIPTEN)
+if (NOT EMSCRIPTEN)
 include (${CMAKE_ROOT}/Modules/TestBigEndian.cmake)
 TEST_BIG_ENDIAN(OPJ_BIG_ENDIAN)
 endif()