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 #28

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

App should treat CFE_SB_MsgId_t values as opaque #28

jphickey opened this issue Apr 6, 2020 · 0 comments · Fixed by #29 or #38
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/to_lab that referenced this issue Apr 6, 2020
@jphickey
Fix nasa#28, Opaque CFE_SB_MsgId_t values
22d7857 
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.
This was referenced Apr 6, 2020
Merged
Closed
jphickey added a commit to jphickey/to_lab that referenced this issue Apr 8, 2020
@jphickey
Fix nasa#28, Opaque CFE_SB_MsgId_t values
f2e853f 
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/to_lab that referenced this issue Apr 8, 2020
@jphickey
Fix nasa#28, Opaque CFE_SB_MsgId_t values
d6b2434 
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/to_lab that referenced this issue May 5, 2020
@jphickey
Fix nasa#28, Opaque CFE_SB_MsgId_t values
a7a6350 
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 that referenced this issue May 11, 2020
@jphickey
Merge pull request #29 from jphickey/fix-28-opaque-msgid
9793ba8 
Fix #28, Opaque CFE_SB_MsgId_t values
Merged manually to resolve conflict
@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