Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

App should treat CFE_SB_MsgId_t values as opaque #45

Closed
jphickey opened this issue Apr 6, 2020 · 0 comments · Fixed by #46 or #47
Closed

App should treat CFE_SB_MsgId_t values as opaque #45

jphickey opened this issue Apr 6, 2020 · 0 comments · Fixed by #46 or #47
Assignees
@jphickey
Labels
enhancement New feature or request
Milestone
Bootes

Comments

@jphickey
Copy link
Contributor

jphickey commented Apr 6, 2020

Is your feature request related to a problem? Please describe.
For compatibility going forward, code should not assume that CFE_SB_MsgId_t is an integer.

Describe the solution you'd like
When dealing with an integer, such as when printing in events/messages or for backward compatibility with MID #define's, the code may use CFE_SB_ValueToMsgId() and CFE_SB_MsgIdToValue() conversion routines.

Additional context
Architecturally, the CFE_SB_MsgId_t is supposed to be an opaque/abstract value that identifies an endpoint on the software bus routing domain. The specific meaning of integer values is already different in an "extended header" (CCSDS v2) build vs. the standard header build. Therefore apps should never make assumptions regarding the specific integer values, and all introspection of CFE_SB_MsgId_t values should be through the CFE SB API only.

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Apr 6, 2020
@jphickey jphickey added the enhancement New feature or request label Apr 6, 2020
jphickey added a commit to jphickey/ci_lab that referenced this issue Apr 6, 2020
@jphickey
Fix nasa#45, Opaque CFE_SB_MsgId_t values
cf78269 
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.
@jphickey jphickey mentioned this issue Apr 6, 2020
Merged
jphickey added a commit to jphickey/ci_lab that referenced this issue Apr 6, 2020
@jphickey
Update nasa#45, Update for formatting
f2c3ac2 
Reduce redability and increase chance of future merge conflicts,
but appease the robo-formatter.
@jphickey jphickey mentioned this issue Apr 6, 2020
Closed
jphickey added a commit to jphickey/ci_lab that referenced this issue Apr 8, 2020
@jphickey
Fix nasa#45, Opaque CFE_SB_MsgId_t values
dae1451 
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.
jphickey added a commit to jphickey/ci_lab that referenced this issue May 5, 2020
@jphickey
Fix nasa#45, Opaque CFE_SB_MsgId_t values
7e36208 
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.
jphickey added a commit to jphickey/ci_lab that referenced this issue May 5, 2020
@jphickey
Fix nasa#45, Opaque CFE_SB_MsgId_t values
1eacca0 
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 added a commit that referenced this issue May 8, 2020
@astrogeco
Merge pull request #46 from jphickey/fix-45-opaque-msgid
c4ec2a7 
Fix #45, Opaque CFE_SB_MsgId_t values
@astrogeco astrogeco mentioned this issue May 13, 2020
Merged
@astrogeco astrogeco added this to the 2.4.0 milestone Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
enhancement New feature or request
Projects
None yet
Milestone
Bootes
2 participants
@jphickey @astrogeco