Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Split up CFE_TBL_Load() and remove early returns #2588

Open
2 tasks done
thnkslprpt opened this issue Aug 9, 2024 · 0 comments · May be fixed by #2589
Open
2 tasks done

Split up CFE_TBL_Load() and remove early returns #2588

thnkslprpt opened this issue Aug 9, 2024 · 0 comments · May be fixed by #2589

Comments

@thnkslprpt
Copy link
Contributor

thnkslprpt commented Aug 9, 2024 •
edited
Loading

Checklist

  • I reviewed the Contributing Guide.
  • I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug

Expected behavior
Split up CFE_TBL_Register, and try to simplify control flow and conditional logic a bit - it currently stands at 223 lines and several dozen combinations of conditional branches...

Also aim to remove/reduce the multiple returns.

Will make testing and future maintenance easier.

Reporter Info
Avi Weiss @thnkslprpt
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Aug 9, 2024
@thnkslprpt
Fix nasa#2588, Split up CFE_TBL_Load and remove early returns
ce52b70
@thnkslprpt thnkslprpt linked a pull request Aug 9, 2024 that will close this issue
Open
2 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #2588, Split up CFE_TBL_Load and remove early returns thnkslprpt/cFE
1 participant
@thnkslprpt