-
Notifications
You must be signed in to change notification settings - Fork 43
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
sample_app Integration candidate: Caelum-rc4+dev64 #214
Conversation
Update internal symbol naming to better follow conventions. Notably, this renames the "SampleAppTable" to be "ExampleTable", among other things. This also adds a separate dispatch source file, splits the unit test into source file units as well.
The display param command shows an example of how to pass in and use a parameter with a command. 3 types are illustrated; a signed and unsigned int, and a string.
…riable Fix #167, Rename CommandCode variable to FcnCode
…f-CFE_TBL_Load-fails Fix #147, Add check for success of CFE_TBL_Load() during Initialization
@@ -73,23 +109,25 @@ | |||
/* This function Process Ground Station Command */ | |||
/* */ | |||
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ | |||
int32 SAMPLE_APP_Process(const SAMPLE_APP_ProcessCmd_t *Msg) | |||
CFE_Status_t SAMPLE_APP_ProcessCmd(const SAMPLE_APP_ProcessCmd_t *Msg) |
Check notice
Code scanning / CodeQL-coding-standard
Long function without assertion Note
/* telemetry, packetize it and send it to the housekeeping task via */ | ||
/* the software bus */ | ||
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ | ||
CFE_Status_t SAMPLE_APP_SendHkCmd(const SAMPLE_APP_SendHkCmd_t *Msg) |
Check notice
Code scanning / CodeQL-coding-standard
Long function without assertion Note
/* command pipe. */ | ||
/* */ | ||
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ | ||
void SAMPLE_APP_TaskPipe(const CFE_SB_Buffer_t *SBBufPtr) |
Check notice
Code scanning / CodeQL-coding-standard
Long function without assertion Note
/* SAMPLE ground commands */ | ||
/* */ | ||
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * **/ | ||
void SAMPLE_APP_ProcessGroundCommand(const CFE_SB_Buffer_t *SBBufPtr) |
Check notice
Code scanning / CodeQL-coding-standard
Long function without assertion Note
/* Verify command packet length */ | ||
/* */ | ||
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * **/ | ||
bool SAMPLE_APP_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLength) |
Check notice
Code scanning / CodeQL-coding-standard
Long function without assertion Note
Checklist (Please check before submitting)
Describe the contribution
Testing performed
cFS Bundle Checks
PSP Checks
Expected behavior changes
See PRs
System(s) tested on
Ubuntu 22.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
@thnkslprpt
@jphickey