Fix MISRA C 2012 Rule 10.8 violation

chinglee-iot commented 1 year ago

Description

MISRA C:2012 Rule 10.8

The value of a composite expression shall not be cast to a different essential type category or a wider essential type.

MISRA violation The following is an example of violation. uxItemSize and uxQueueLength is of UBaseType_t. The multiply result is still UBaseType_t. Convert UBaseType_t to size_t may cause implicit type conversion.

UBaseType_t uxQueueLength, uxItemSize;

xQueueSizeInBytes = ( size_t ) ( uxQueueLength * uxItemSize );

Fix In this PR

Update for composite expression of size_t type variables.

Test Steps

N/A

Checklist:

[ ] ~~I have tested my changes. No regression in existing tests.~~
[ ] ~~I have modified and/or added unit-tests to cover the code changes in this Pull Request.~~

Related Issue

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

codecov[bot] commented 1 year ago

Codecov Report

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

:exclamation: No coverage uploaded for pull request base (main@1813a45). Click here to learn what that means.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #853 +/- ## ======================================= Coverage ? 93.64% ======================================= Files ? 6 Lines ? 3179 Branches ? 885 ======================================= Hits ? 2977 Misses ? 95 Partials ? 107 ``` | [Flag](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/853/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/853/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | `93.64% <100.00%> (?)` | | 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.