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

Test command message should use CFE_SB_CmdHdr_t #74

Closed
skliper opened this issue Jun 9, 2020 · 1 comment
Closed

Test command message should use CFE_SB_CmdHdr_t #74

skliper opened this issue Jun 9, 2020 · 1 comment
Labels
invalid This doesn't seem right

Comments

@skliper
Copy link
Contributor

skliper commented Jun 9, 2020

Is your feature request related to a problem? Please describe.
TestMsg uses CFE_SB_Msg_t, where it should use CFE_SB_CmdHdr_t for a zero parameter command

Describe the solution you'd like
Switch to CFE_SB_CmdHdr_t

Describe alternatives you've considered
None

Additional context
Allows the changes being worked in nasa/cFE#726

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added the enhancement New feature or request label Jun 9, 2020
@skliper skliper added this to the 1.2.0 milestone Jun 9, 2020
@skliper skliper added invalid This doesn't seem right and removed enhancement New feature or request labels Jun 9, 2020
@skliper skliper removed this from the 1.2.0 milestone Jun 9, 2020
@skliper
Copy link
Contributor Author

skliper commented Jun 9, 2020

Retracted - unit test just passes in actual size to check against actual size.

@skliper skliper closed this as completed Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@skliper