Revert PR #52

[dzbaker]

Checklist (Please check before submitting)

• I reviewed the Contributing Guide.
• I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
The changes included in PR #52 introduce bugs. We will revert the PR for this release, then revisit it following release.

Reporter Info
Dylan Z. Baker - NASA/GSFC

[skliper]

@dzbaker - what were the bugs? Documented anywhere?

[dmknutsen]

The change was causing issues with functional testing because the number of characters used (2) in the following calculation is incorrect:
#define MM_MAX_DUMP_INEVENT_BYTES ((CFE_MISSION_EVS_MAX_MESSAGE_LENGTH - (13 + 33)) / 2)

The value should actually be 3 because there is a space in between each byte displayed. The end result is that the message gets truncated and the memory location dumped gets corrupted/lost. Because we are wrapping up the build/past the cut off, and the change is relatively minor, we are trying to punt this to the next build.

Good catch on the documentation - we need to clean up the PR/Issue to make sure we don't accidentally drop this next round.