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

Update app dev guide and unit tests to use CFE_MSG_PTR instead of &(x).Msg #2057

Closed
skliper opened this issue Feb 23, 2022 · 1 comment · Fixed by #2472
Closed

Update app dev guide and unit tests to use CFE_MSG_PTR instead of &(x).Msg #2057

skliper opened this issue Feb 23, 2022 · 1 comment · Fixed by #2472
Labels
docs This change only affects documentation. unit-test

Comments

@skliper
Copy link
Contributor

skliper commented Feb 23, 2022

Is your feature request related to a problem? Please describe.
&(x).Msg still used in UT and docs where CFE_MSG_PTR should be used to avoid dependency on internal CFE_MSG_Message_t elements.

Describe the solution you'd like
Replace with CFE_MSG_PTR

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added docs This change only affects documentation. unit-test labels Feb 23, 2022
@skliper
Copy link
Contributor Author

skliper commented Feb 23, 2022

Not backwards compatible w/ Caelum... not targeting Draco.

thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Dec 5, 2023
@thnkslprpt
Fix nasa#2057, Update docs and UT to use CFE_MSG_PTR
0cce898
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Dec 6, 2023
@thnkslprpt
Fix nasa#2057, Update docs and UT to use CFE_MSG_PTR
6a57ec2
@thnkslprpt thnkslprpt mentioned this issue Dec 6, 2023
Merged
2 tasks
dzbaker added a commit that referenced this issue Dec 12, 2023
@dzbaker
Merge pull request #2472 from thnkslprpt:fix-2057-update-docs-and-UT-…
68da2a7 
…with-CFE_MSG_PTR

Fix #2057, Update docs and UT to use CFE_MSG_PTR
@dzbaker dzbaker mentioned this issue Dec 12, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs This change only affects documentation. unit-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #2057, Update docs and UT to use CFE_MSG_PTR thnkslprpt/cFE
1 participant
@skliper