xioTechnologies/Fusion

Bug - Commit "Use designated initialisers" breaks compilation in C++

Closed this issue · 5 comments

ladislas commented

Downloading the latest source from main, it seems that ec50751 introduced a bug when working in C++.

image

xioTechnologies commented

Please can you confirm which compiler you are using. Please can you also show me how you include Fusion in your .cpp files. You should only use #include "Fusion.h because this file wraps the library in extern "C".

ladislas commented

Please can you confirm which compiler you are using

arm-none-eabi-gcc --version
arm-none-eabi-gcc (GNU Arm Embedded Toolchain 10.3-2021.10) 10.3.1 20210824 (release)
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Please can you also show me how you include Fusion in your .cpp files

Here: https://github.com/leka/LekaOS/blob/ladislas/feature/replace-mahony-with-new-fusion/libs/IMUKit/source/IMUKit.cpp#L13

xioTechnologies commented

We are investigating this. In the mean time, please either use the latest version, or revert ec50751 to avoid the issue.

xioTechnologies commented

The issue should be fixed as of dde922f. Please can you confirm the fix on your system.

ladislas commented

I can confirm that the issue has been fixed, thanks a lot!

  • 👍1