MISRA fixes

[bradleysmith23]

Description

MISRA violations identified through the use of Coverity and here
Fixes to violations of MISRA rule 10.3 and rule 20.12
MISRA rule 10.3 violations: assigning expression of essentially unsigned type to an object with essentially signed type. (int32_t value = 0U;) and assignment of int8_t to a char (char_ c = { 0 }; and char_ n = { 0 };)

MISRA rule 20.12 violation: assert( isOpenBracket_( mode ) ); -> Here, macro parameter isOpenBracket_ (which is also macro) is itself subject for further macro replacement (#expr), supplied as expr parameter.

Test Steps

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.

[bradleysmith23]

/bot run formatting

[bradleysmith23]

/bot run formatting

[paulbartell]

It's not clear why we need to change this. There must be a better way to satisfy the customer's MISRA checker.

[bradleysmith23]

/bot run formatting

[chinglee-iot]

MISRA C 2012 rule 20.12 deviation doesn't seem to be fixed when #define configJSON_ASSERT( expr ) assert( exprt )
Creating a new PR #157 to cover this PR.