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

cFE Integration candidate: Equuleus-rc1+dev5 #2518

Merged
merged 5 commits into from
Feb 23, 2024
Merged

Conversation

dzbaker
Copy link
Collaborator

@dzbaker dzbaker commented Feb 22, 2024

Checklist (Please check before submitting)

Describe the contribution

Testing performed
cFS Bundle Checks
cFE Checks

Expected behavior changes
See PRs

System(s) tested on
Ubuntu 20.04

Additional context
Add any other context about the contribution here.

Third party code
If included, identify any third party code and provide text file of license

Contributor Info - All information REQUIRED for consideration of pull request
@jphickey
@thnkslprpt

thnkslprpt and others added 5 commits July 18, 2023 10:10
Having this abstract type as a "struct" makes it match the Command
and Telemetry abstract types.  Futhermore, it better conveys the
intent that this is an abstract object and should not be directly
used or accessed in other ways.

It may still be implemented as a union underneath (depending on
how MSG module chooses to implement) but that is hidden from
public API.  In the case of the default MSG module implementation,
there are just a handful of cases where it is accessed internally
as bytes, and those are simple enough to do with a cast.
Fix #2514, change CFE_MSG_Message from union to struct
…in-CFE_ES_RunPerfLogDump

Fix #1933, Remove unreachable/dead branch in CFE_ES_RunPerfLogDump()
@dzbaker dzbaker force-pushed the integration-candidate branch from 4464c66 to 110fa46 Compare February 23, 2024 13:06
@dzbaker dzbaker merged commit 9e5d452 into main Feb 23, 2024
37 of 38 checks passed
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.

3 participants