Fix #216, #227 - Adds strict cppcheck and fixes remaining warnings (replace codec macros)

[skliper]

Checklist (Please check before submitting)

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License agreement to [email protected] and copied [email protected].

Describe the contribution

• Fix Resolve cppcheck issues (and confirm they are getting flagged correctly) #216
• Fix Replace codec code-compatibility macros with direct calls to the new functions #227

Testing performed
Build/run unit tests, also manually ran strict cppcheck on fsw/

Expected behavior changes
None except now enforcing strict cppcheck in CI

System(s) tested on

• Hardware: i5/wsl
• OS: Ubuntu 18.04
• Versions: Bundle main + this commit

Additional context
Workflow will fail until #226 is merged since one of the warnings were fixed there.
Also will fail until nasa/cFS#454 is merged based on the directory fix

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

[astrogeco]

CCB:2022-04-13 APPROVED

[astrogeco]

Resolved conflict and added hotfixes to fix cppcheck error quoted below. See fix in astrogeco@96cdf30

[fsw/src/cf_codec.h:821] -> [fsw/src/cf_codec.c:1050]: (style, inconclusive) Function 'CF_CFDP_DecodeCrc' argument 2 names different: declaration 'pcrc' definition 'plcrc'.
Error: Process completed with exit code 255.