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_EVS_Send{Crit|Critical|Err|Error|Info|Information|Debug} wrapper for CFE_EVS_SendEvent #788

Closed
CDKnightNASA opened this issue Jul 31, 2020 · 0 comments · Fixed by #867 or #1008
Assignees
Milestone

Comments

@CDKnightNASA
Copy link
Contributor

Is your feature request related to a problem? Please describe.
With the expansion of the CFE_EVS_EventType_* constants as part of CFE_EVS_SendEvent(), it got me thinking that we have a lot of extraneous typing when there's only four types of events. What about adding wrapper macros/functions that have the "EventType" as part of the function name? I also recommend both long and short versions, such as CFE_EVS_SendCrit() and CFE_EVS_SendCritical().

So, for example:

CFE_EVS_SendEvent(MYAPP_EVENT_EID, CFE_EVS_EventType_ERROR, "error message (%d)", status);

becomes

CFE_EVS_SendErr(MYAPP_EVENT_EID, "error message (%d)", status);

Requester Info
Christopher.D.Knight@nasa.gov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants