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 #2514, change CFE_MSG_Message from union to struct #2515

Merged
merged 1 commit into from
Feb 23, 2024

Commits on Feb 13, 2024

  1. Fix nasa#2514, change CFE_MSG_Message from union to struct

    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.
    jphickey committed Feb 13, 2024
    Configuration menu
    Copy the full SHA
    777feec View commit details
    Browse the repository at this point in the history