You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Other instances in OS_TimeFromTotalSeconds, OS_TimeFromTotalMilliseconds, OS_TimeFromTotalMicroseconds, OS_TimeFromTotalNanoseconds. Issue seems to have been introduced in #1310 .
I will look at this. For now, if you wanted to patch your code, just remove the .ticks = designation part, it should work fine (there is just one member in this wrapper object).
We don't currently validate the OSAL headers for use in C++ at all, we should re-evaluate if this is something we should consider adding to our workflows. I know there are a handful of C++ users out there.
Describe the bug
Designated initializer in
osapi-clock.h
header breaks C++ compilation unit. C++ does not support this until C++20To Reproduce
#include "cfe.h"
in a C++ compilation unitBuild file, observe error.
Expected behavior
Successful compilation
Code snips
Instance of designated initializer used here:
osal/src/os/inc/osapi-clock.h
Line 253 in fa9671f
Other instances in
OS_TimeFromTotalSeconds
,OS_TimeFromTotalMilliseconds
,OS_TimeFromTotalMicroseconds
,OS_TimeFromTotalNanoseconds
. Issue seems to have been introduced in #1310 .System observed on:
Additional context
Inline header functions should use C++03 (latest standard supported by VxWorks 6.9) compatible constructs.
Reporter Info
John N Pham, Northrop Grumman
The text was updated successfully, but these errors were encountered: