Skip to content

Integration Candidate COMBINED 2020-04-29 and 2020-05-06#47

Merged
astrogeco merged 3 commits into
masterfrom
integration-candidate
May 13, 2020
Merged

Integration Candidate COMBINED 2020-04-29 and 2020-05-06#47
astrogeco merged 3 commits into
masterfrom
integration-candidate

Conversation

@astrogeco

@astrogeco astrogeco commented May 8, 2020

Copy link
Copy Markdown
Contributor

Describe the contribution
Fixes #45

Testing performed
See PRs
Bundle CI: https://travis-ci.com/github/nasa/cFS/builds/165726730

Expected behavior changes
PR #46 - 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.

System(s) tested on
See PRs

Additional context
Part of nasa/cFS#83

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

jphickey and others added 2 commits May 5, 2020 16:54
Do not assume CFE_SB_MsgId_t is implicitly integral in nature.
When an integer value is required for printing or backward
compatibility, use the explicit conversion routine to
get this.
@astrogeco astrogeco changed the title Integration Candidate: 2020-04-29 Integration Candidate COMBINED 2020-04-29 and 2020-05-06 May 13, 2020
@astrogeco astrogeco marked this pull request as ready for review May 13, 2020 01:49
Update README by adding version history section and tidying up markdown.
@astrogeco astrogeco merged commit a589163 into master May 13, 2020
@skliper skliper added this to the 2.4.0 milestone Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App should treat CFE_SB_MsgId_t values as opaque

3 participants