nasa / sample_app

The Core Flight System (cFS) Sample App (sample_app)
Apache License 2.0
49 stars 43 forks source link

Symbol name cleanup and remainder of pattern/convention fixes #210

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Is your feature request related to a problem? Please describe. There are still a few symbol names in sample_app that do not (quite) adhere to the conventions, and some further source file organization changes.

Examples - SampleAppTable vs. SAMPLE_APP_Table_t - Type name vs. Table name mismatched. Command processor functions should end in Cmd suffix There should be separate "dispatch" file for the TaskPipe function, that switches based on received msgID and CC.

Describe the solution you'd like Rename symbols, organize source functions.

Also - The unit test should be split into units per source file - this is a preferred approach for larger apps, so SAMPLE_APP should use this template, even though it is simple and thus not strictly necessary

Additional context This should bring sample_app fully into compliance

Requester Info Joseph Hickey, Vantage Systems, Inc.