Open ladislas opened 1 year ago
develop
/ 6fc6b93d + mbed-os@master+fixes+gcc-11-support
+ -std=c++20
ladislas/feature/compiler-add-warnings
/ b816730b + mbed-os@master+fixes+gcc-11-support
+ -std=c++20
GNU Arm Embedded Toolchain 10.3-2021.10 - 10.3.1 20210824
OFF
Target | Flash Used (base/head) | Fash Used Δ | Flash Available (base/head) | Static RAM (base/head) | Static RAM Δ |
---|---|---|---|---|---|
bootloader | 170784 (65%) | ø | 91360 (34%) 91360 (34%) |
30968 (5%) | ø |
os | 435024 (27%) 434960 (27%) |
:chart_with_downwards_trend: -64 (0%) |
1129648 (72%) 1129712 (72%) |
83112 (15%) 83104 (15%) |
:chart_with_downwards_trend: -8 (0%) |
LekaOS
(click to expand)functional_ut_core_imu
(click to expand)functional_ut_imu_kit
(click to expand)spike_lk_accel_gyro
(click to expand)spike_lk_command_kit
(click to expand)spike_lk_imu_kit
(click to expand)spike_lk_motion_kit
(click to expand)spike_lk_reinforcer
(click to expand)develop
/ 6fc6b93d + mbed-os@master+fixes+gcc-11-support
+ -std=c++20
ladislas/feature/compiler-add-warnings
/ b816730b + mbed-os@master+fixes+gcc-11-support
+ -std=c++20
GNU Arm Embedded Toolchain 10.3-2021.10 - 10.3.1 20210824
ON
Target | Flash Used (base/head) | Fash Used Δ | Flash Available (base/head) | Static RAM (base/head) | Static RAM Δ |
---|---|---|---|---|---|
bootloader | 183068 (69%) | ø | 79076 (30%) 79076 (30%) |
41664 (7%) | ø |
os | 495996 (31%) | ø | 1068676 (68%) 1068676 (68%) |
94328 (17%) 94320 (17%) |
:chart_with_downwards_trend: -8 (0%) |
LekaOS
(click to expand)functional_ut_core_imu
(click to expand)functional_ut_imu_kit
(click to expand)spike_lk_accel_gyro
(click to expand)spike_lk_command_kit
(click to expand)spike_lk_imu_kit
(click to expand)spike_lk_motion_kit
(click to expand)spike_lk_reinforcer
(click to expand)Merging #1237 (b816730) into develop (6fc6b93) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## develop #1237 +/- ##
========================================
Coverage 96.21% 96.21%
========================================
Files 150 150
Lines 3727 3730 +3
========================================
+ Hits 3586 3589 +3
Misses 141 141
Impacted Files | Coverage Δ | |
---|---|---|
drivers/CoreIMU/source/CoreLSM6DSOX.cpp | 95.08% <100.00%> (ø) |
|
drivers/CoreVideo/include/CGColor.hpp | 100.00% <100.00%> (ø) |
|
libs/ContainerKit/include/CircularQueue.h | 100.00% <100.00%> (ø) |
|
libs/LogKit/include/LogKit.h | 100.00% <100.00%> (ø) |
|
libs/Utils/include/MathUtils.h | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
@YannLocatelli
There are some more warnings
Yes, I started by focusing on unit tests first
in
spikes/lk_behavior_kit/main.cpp:166:19 spikes/lk_command_kit/main.cpp:146:18 spikes/lk_fs/main.cpp:79:18 spikes/lk_lcd/main.cpp:59:17 spikes/lk_reinforcer/main.cpp:182:18 app/os/main.cpp:251:18
cannot be "fixed" but silenced if needed.
tests/functional/include/tests/utils_sleep.h:61:68
can be silenced with a comment saying that the code is from mbed-os and kept as is for the moment
libs/Utils/source/MathUtils.cpp:50:7 libs/Utils/source/MathUtils.cpp:53:7
can be silenced as well as we are using "official" code implementation for the fast invert square root
that being said, this code will be removed in the future when we start using the new fusion algorithms which ship their own implementation
spikes/lk_log_kit/main.cpp:47:7
I'll look into it, but it's done for testing
- | Version | Same as filename | Same as os_version |
---|---|---|---|
os | 1.3.0+1674489231 |
✔️ | ✔️ |
firmware (os + bootloader) |
1.3.0+1674489231 |
✔️ | ✔️ |
Kudos, SonarCloud Quality Gate passed!