Closed thnkslprpt closed 1 year ago
Checklist
Describe the bug md_app.c contains a couple of consecutive, mutually exclusive status checks.
md_app.c
Code snips https://github.com/nasa/MD/blob/116d2986500101551beb32bcd3fa30de4cfd39b9/fsw/src/md_app.c#L188-L197 https://github.com/nasa/MD/blob/116d2986500101551beb32bcd3fa30de4cfd39b9/fsw/src/md_app.c#L274-L283
Expected behavior Combine into an if/else - no need to do 2 evaluations.
if
else
Reporter Info Avi Weiss @thnkslprpt
Checklist
Describe the bug
md_app.c
contains a couple of consecutive, mutually exclusive status checks.Code snips https://github.com/nasa/MD/blob/116d2986500101551beb32bcd3fa30de4cfd39b9/fsw/src/md_app.c#L188-L197 https://github.com/nasa/MD/blob/116d2986500101551beb32bcd3fa30de4cfd39b9/fsw/src/md_app.c#L274-L283
Expected behavior Combine into an
if
/else
- no need to do 2 evaluations.Reporter Info Avi Weiss @thnkslprpt