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
Currently there are issues when building pico with c++ compiler:
All the public api are wrapped in extern "C", while none of the internal code is - which can lead to ub.
We are using compound initializers inside internal code - which are not part of C++ standard (even designated initializers are only supported since C++ 20)
It would be good to address this issues.
The first one can be probably fixed by setting a config variable Z_USING_CPP_COMPILER (or something similar) based on the platform/ compiler info from cmake.
To solve the second point, we would need to correct all uses of compound initializers (we can leave designated initializers if we state that we require at least C++ 20 compliant compiler).
The text was updated successfully, but these errors were encountered:
Describe the release item
Currently there are issues when building pico with c++ compiler:
It would be good to address this issues.
The first one can be probably fixed by setting a config variable Z_USING_CPP_COMPILER (or something similar) based on the platform/ compiler info from cmake.
To solve the second point, we would need to correct all uses of compound initializers (we can leave designated initializers if we state that we require at least C++ 20 compliant compiler).
The text was updated successfully, but these errors were encountered: