Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix #179, Light initialization logic refactor + remove multiple returns #180

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).
Tested locally with cFS build + test commands - all working fine.

Expected behavior changes

  • if CFE_SB_CreatePipe fails when creating the command pipe, to_lab will not go on try to create the telemetry pipe.
  • if CFE_SB_CreatePipe fails when creating the telemetry pipe, to_lab will not try to add the subscriptions - which makes little sense anyway...
  • also, in neither case will to_lab now report successful initialization if the pipes cannot be created.
  • the delete handler will be installed now in all cases even if one of the early calls to cFE fails - this is the case also in other apps.

System(s) tested on
Debian GNU/Linux 11 (bullseye)
Current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

@thnkslprpt thnkslprpt force-pushed the fix-179-initialization-logic-improvements branch from cc850eb to e8f3dcf Compare March 23, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Light initialization logic refactor + remove multiple returns
1 participant