FreeRTOS / FreeRTOS-Kernel

FreeRTOS kernel files only, submoduled into https://github.com/FreeRTOS/FreeRTOS and various other repos.
https://www.FreeRTOS.org
MIT License
2.62k stars 1.09k forks source link

Fix MISRA C 2012 rule 7.2 violation for stack macro #921

Closed chinglee-iot closed 8 months ago

chinglee-iot commented 8 months ago

The original rule 7.2 fix disappeared during the merge process.

Fix MISRA C 2012 rule 7.2 violation

Description

MISRA C 2012 rule 7.2

A "u" or "U" suffix shall be applied to all integer constants that are represented in an unsigned type.

MISRA violation

MISRA enforces the use of the U or u suffix for unsigned integer constants. The statement which leads to the violation const uint32_t ulCheckValue = ( uint32_t ) 0xa5a5a5a5;

Test Steps

N/A.

Checklist:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

sonarcloud[bot] commented 8 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5dbfd38) 93.78% compared to head (6671954) 93.78%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #921 +/- ## ======================================= Coverage 93.78% 93.78% ======================================= Files 6 6 Lines 3184 3184 Branches 885 885 ======================================= Hits 2986 2986 Misses 91 91 Partials 107 107 ``` | [Flag](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/921/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/921/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | `93.78% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.