nasa / cFE

The Core Flight System (cFS) Core Flight Executive (cFE)
Apache License 2.0
402 stars 198 forks source link

Remove conditionally-compiled code in TBL #2545

Open jphickey opened 3 months ago

jphickey commented 3 months ago

Is your feature request related to a problem? Please describe. There are some code chunks that are within #if blocks, based on the configuration parameters CFE_PLATFORM_TBL_VALID_SCID_COUNT and CFE_PLATFORM_TBL_VALID_PRID_COUNT.

Describe the solution you'd like Refactor code so it is not conditionally compiled. (limit preprocessor logic usage to data structure initializers, not runtime instructions)

Describe alternatives you've considered N/A

Additional context Conditionally-compiled code is not in accordance with current coding standards. Runtime loops of 0 size are OK. The unit test/coverage test must be able to execute the code, which is not possible if it has been compiled out.

Requester Info Joseph Hickey, Vantage Systems, Inc.