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.

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

Fix #123, use CFE_MSG_PTR conversion macro #124

Closed
wants to merge 1 commit into from

Conversation

jphickey
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
Rather than taking the address of a sub-member, use the macro provided from the CFE_MSG module to convert from a local container to a CFE_MSG_Messge_t pointer.

Fixes #123

Testing performed
Build and run all tests

Expected behavior changes
None.

System(s) tested on
Debian

Additional context
This makes the source code more compatible with alternative implementations of the CFE_MSG module (e.g. if sub-structs are not named identically).

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

Rather than taking the address of a sub-member, use the macro provided
from the CFE_MSG module to convert from a local container to a
CFE_MSG_Messge_t pointer.
@jphickey
Copy link
Contributor Author

Superseded by #125 (dependency issue)

@jphickey jphickey closed this Oct 24, 2023
@jphickey jphickey deleted the fix-123-msgptr branch October 26, 2023 20:12
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.

Use CFE_MSG_PTR conversion macro
1 participant