-
Notifications
You must be signed in to change notification settings - Fork 29
Open
Labels
Description
Checklist (Please check before submitting)
- I reviewed the Contributing Guide.
- I reviewed the README file to see if the feature is in the major future work.
- I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.
Is your feature request related to a problem? Please describe.
For the standard 10 apps, SC is the only one to use the CFE_TBL_NotifyByMessage API and manage tables on command.
Lines 606 to 612 in 68e2520
| void SC_RegisterManageCmds(void) | |
| { | |
| int32 i; | |
| /* Register for RTS info table manage request commands */ | |
| CFE_TBL_NotifyByMessage(SC_OperData.RtsInfoHandle, CFE_SB_ValueToMsgId(SC_CMD_MID), SC_MANAGE_TABLE_CC, | |
| SC_TBL_ID_RTS_INFO); |
Lines 683 to 685 in 68e2520
| case SC_MANAGE_TABLE_CC: | |
| SC_TableManageCmd(BufPtr); | |
| break; |
Lines 721 to 731 in 68e2520
| void SC_TableManageCmd(const CFE_SB_Buffer_t *BufPtr) | |
| { | |
| int32 ArrayIndex; | |
| int32 TableID = (int32)((CFE_TBL_NotifyCmd_t *)BufPtr)->Payload.Parameter; | |
| /* Manage selected table as appropriate for each table type */ | |
| if ((TableID >= SC_TBL_ID_ATS_0) && (TableID < (SC_TBL_ID_ATS_0 + SC_NUMBER_OF_ATS))) | |
| { | |
| ArrayIndex = TableID - SC_TBL_ID_ATS_0; | |
| SC_ManageAtsTable(ArrayIndex); | |
| } |
Describe the solution you'd like
Could instead just periodically manage all the tables (like all the rest).
Describe alternatives you've considered
None
Additional context
None
Requester Info
Jacob Hageman - NASA/GSFC