-
Notifications
You must be signed in to change notification settings - Fork 215
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 #1107, Remove unreachable branch (superfluous if condition) #1368
Conversation
Does this change branch coverage results at all? If so I recommend updating:
|
19dc2da
to
a357ce4
Compare
a357ce4
to
ea2faf7
Compare
Thanks! I need a checklist or something - that's the second time I forgot to update these parameters for allowed exceptions to coverage. Anyway I updated it now - it had actually become out-of-date at some point before this PR (was allowing 4 exceptions when the main branch was already down to only 3 missed branches). I've lowered it now from 4 to 2. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I remember correctly, this could be a remnant from the days when there was a global volume table in OSAL that could pre-init these items. I think we've fully purged that, and I concur I can't see any path though the code where this condition could ever be false.
*Combines:* cFE equuleus-rc1+dev84 osal equuleus-rc1+dev41 **Includes:** *cFE* - nasa/cFE#2505 *osal* - nasa/osal#1368 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
*Combines:* cFE equuleus-rc1+dev84 osal equuleus-rc1+dev41 **Includes:** *cFE* - nasa/cFE#2505 *osal* - nasa/osal#1368 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
Checklist
Describe the contribution
OS_FileSys_Initialize
#1107if
condition removed - removing an unreachable branch (condition is alwaystrue
) and slightly improving test coverage.Testing performed
GitHub CI actions (incl. Build + Run, Unit Tests etc.) all passing successfully.
Expected behavior changes
No change to behavior.
Contributor Info
Avi Weiss @thnkslprpt