COVESA/capicxx-someip-runtime

New version 3.2.3 does not compiles on windows (__BYTE_ORDER_ not defined)

Opened this issue · 1 comments

Following the latest update (d346e0f) the compilation does not work on windows (msvc 2019)
I got compilation errors on include/CommonAPI/SomeIP/OutputStream.hpp (_BYTE_ORDER not defined)
d346e0f#diff-4c81b6e7390e799245076afd2b9125066fb6b2c4f727949f4bc5cc46f633c0bfR570-R595

For some reason, code has been factorised and is now not compiling.
gcc seems to allow usage of compiler definition in normal if/else condition, but msvc does not allow it.

The compilation works fine after reverting lines 570-595 of OutputStream.hpp (same functional behaviour)

hello @AlexisCompain
thanks for the input.
We will check internally and solve this in the next version.

cc @DiogoPedrozza @dibpinto @fcmonteiro