We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Checklist
Describe the bug HS_EnableEventMonCmd() and HS_DisableEventMonCmd() have consecutive, mutually exclusive status checks.
HS_EnableEventMonCmd()
HS_DisableEventMonCmd()
Code snips
HS/fsw/src/hs_cmds.c
Lines 409 to 416 in 76915e9
Lines 465 to 472 in 76915e9
Expected behavior Combine into an if/else - no need to do 2 evaluations.
if
else
Reporter Info Avi Weiss @thnkslprpt
The text was updated successfully, but these errors were encountered:
Fix nasa#77, Combine consecutive, mutually-exclusive status checks
49f95bc
43a17cf
Merge pull request #78 from thnkslprpt/fix-77-combine-consecutive-mut…
7697f3f
…ually-exclusive-status-checks Fix #77, Combine consecutive, mutually-exclusive status checks
Successfully merging a pull request may close this issue.
Checklist
Describe the bug
HS_EnableEventMonCmd()
andHS_DisableEventMonCmd()
have consecutive, mutually exclusive status checks.Code snips
HS/fsw/src/hs_cmds.c
Lines 409 to 416 in 76915e9
HS/fsw/src/hs_cmds.c
Lines 465 to 472 in 76915e9
Expected behavior
Combine into an
if
/else
- no need to do 2 evaluations.Reporter Info
Avi Weiss @thnkslprpt
The text was updated successfully, but these errors were encountered: