cFE icon indicating copy to clipboard operation
cFE copied to clipboard

Fix #2627, update check, docstr for max table sz

Open irowebbn opened this issue 1 year ago • 2 comments

Checklist (Please check before submitting)

Describe the contribution A clear and concise description of what the contribution is.

  • Fixes #2627 by updating doctring in example config and updating preprocessor verification.

Testing performed Steps taken to test the contribution:

  1. Normal build and compile steps

Expected behavior changes A clear and concise description of how this contribution will change behavior and level of impact.

  • Allows a wider range of combinations for table size and table memory.

System(s) tested on

  • Versions: cFE trunk

Contributor Info - All information REQUIRED for consideration of pull request Isaac Rowe NASA JSC, JETS II contract

irowebbn avatar Jan 23 '25 15:01 irowebbn

I understand that there is a new method to configure cFE that obviates the need for the headers. I was unable to find documentation on how to setup and run that. This PR does not address any necessary changes that would be needed for that method.

irowebbn avatar Jan 23 '25 15:01 irowebbn

Also, this change implies that MAX_SIMULTANEOUS_LOADS can be set to 1. However, doing so breaks some old unit tests. I can address that in a follow-up PR.

irowebbn avatar Jan 23 '25 15:01 irowebbn