Fix #45, Opaque CFE_SB_MsgId_t values

[jphickey]

Describe the contribution
Apply the CFE_SB_MsgIdToValue() and CFE_SB_ValueToMsgId() routines where compatibility with an integer MsgId is necessary - syslog prints, events, compile-time MID #define values.

Fixes #45

Testing performed
Unit test
Execute CFE and sanity-check normal operation - send commands to app using cmdUtil and confirm commands are processed normally.

Expected behavior changes
No impact to behavior.

System(s) tested on
Ubuntu 18.04 LTS, 64-bit

Additional context
In future versions of CFE SB the MsgId type might not be a simple integer, so this is one step in the direction of avoiding this assumption in apps.

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

[jphickey]

Note - the robo-formatter apparently insists that we shouldn't put one argument per line, even though this is much more readable that way:

-                          (unsigned int)CFE_SB_MsgIdToValue(MessageID),
-                          (unsigned int)CommandCode, 
-                          (unsigned int)ActualLength, 
-                          (unsigned int)ExpectedLength);
+                          (unsigned int)CFE_SB_MsgIdToValue(MessageID), (unsigned int)CommandCode,
+                          (unsigned int)ActualLength, (unsigned int)ExpectedLength);

Does the CCB really think that the robo-formatter is better here? In this case (and one other) I had intentionally put one argument per line because it is more readable.

[jphickey]

Pushed simplified changeset that corresponds with other apps - see description here: nasa/sample_app#54 (comment)

[astrogeco]

Related to nasa/cFE#592

[astrogeco]

CCB 20200408 - Split up into parts.

[astrogeco]

Marking based on CCB 20200408

[jphickey]

Update to minimal change to correlate with CFE. Required to resolve compile error when building RTEMS with extended headers.

[astrogeco]

CCB 20200506 - APPROVED