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

Apps should use CFE_MSG_PTR macro instead of cast or local unwrapping #19

Closed
skliper opened this issue Apr 22, 2022 · 0 comments · Fixed by #85
Closed

Apps should use CFE_MSG_PTR macro instead of cast or local unwrapping #19

skliper opened this issue Apr 22, 2022 · 0 comments · Fixed by #85
Labels
enhancement

Comments

@skliper
Copy link
Contributor

skliper commented Apr 22, 2022

Apps typically cast to a CFE_MSG_Message_t or use *.Msg. Better to use abstracted CFE_MSG_PTR. NOTE - Not backwards compatible with Caelum so recommend not addressing in Draco.

Imported from GSFCCFS-1900
jphickey added a commit to jphickey/LC that referenced this issue Mar 21, 2023
@jphickey
Fix nasa#19, Use CFE_MSG_PTR macro and other name updates
4b280d9 
Follow naming conventions for cmd/tlm data structures and use provided
macros where appropriate.

This also regenerates the stubs, and are committed unmodified from the
generator tool.  The global data object is split to a separate
compilation unit so as to not conflict with the generated files.
jphickey added a commit to jphickey/LC that referenced this issue Mar 28, 2023
@jphickey
Fix nasa#19, Use CFE_MSG_PTR macro and other name updates
7f3c384 
Follow naming conventions for cmd/tlm data structures and use provided
macros where appropriate.

This also regenerates the stubs, and are committed unmodified from the
generator tool.  The global data object is split to a separate
compilation unit so as to not conflict with the generated files.
jphickey added a commit to jphickey/LC that referenced this issue Mar 29, 2023
@jphickey
Fix nasa#19, Use CFE_MSG_PTR macro and other name updates
d6a7f83 
Follow naming conventions for cmd/tlm data structures and use provided
macros where appropriate.

This also regenerates the stubs, and are committed unmodified from the
generator tool.  The global data object is split to a separate
compilation unit so as to not conflict with the generated files.
jphickey added a commit to jphickey/LC that referenced this issue Mar 29, 2023
@jphickey
Fix nasa#19, Use CFE_MSG_PTR macro and other name updates
7907fd4 
Follow naming conventions for cmd/tlm data structures and use provided
macros where appropriate.
@jphickey jphickey mentioned this issue Mar 29, 2023
Merged
2 tasks
jphickey added a commit to jphickey/LC that referenced this issue Mar 29, 2023
@jphickey
Fix nasa#19, Use CFE_MSG_PTR macro and other name updates
4373781 
Follow naming conventions for cmd/tlm data structures and use provided
macros where appropriate.
dzbaker added a commit that referenced this issue Mar 30, 2023
@dzbaker
Merge pull request #85 from jphickey/fix-19-use-msg-ptr
5554bd1 
Fix #19, Use CFE_MSG_PTR macro and other name updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #19, Use CFE_MSG_PTR macro and other name updates jphickey/LC
1 participant
@skliper