search

cfs-tools / cfs-basecamp

Provides a framework and tools for developing, downloading, and integrating core Flight System (cFS) applications into an operational system.
Other
24 stars 2 forks source link

Improve APP_C_DEMO table load command's logic #56

Closed dmccomas closed 5 months ago

dmccomas commented 9 months ago

The table acceptance callback function does not return any status, so the table is always accepted. if (HistogramTbl->LoadFunc != NULL) (HistogramTbl->LoadFunc)();

The table load command sets a local 'LastLoadStatus' that should be removed. I think this design was prior to TBLMGR enhancements and the addition of TBLMGR_GetLastTblStatus() which is used by the app's SendStatusTelemetry(). Once this is solved for APP_C_DEMO the other Basecamp apps should be analyzed. HistogramTbl->LastLoadStatus = TBLMGR_STATUS_VALID;