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

Fix #108, Remove multiple returns from `HS_AppInit()`, `HS_SbInit()` and `HS_TblInit()` #109

Open

thnkslprpt wants to merge 1 commit into nasa:main from thnkslprpt:fix-108-factor-out-multiple-returns

Contributor

thnkslprpt commented Jun 18, 2023

Checklist

I reviewed the Contributing Guide.
I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution

Fixes Factor out multiple returns from HS_AppInit(), HS_SbInit() and HS_TblInit() #108
- HS_AppInit(), HS_SbInit() and HS_TblInit() refactored to have a single exit point

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Coverage Tests etc.).

Expected behavior changes
No change to function logic/behavior other than removing the early returns.

Contributor Info
Avi Weiss @thnkslprpt

chillfig added the enhancement label


          Fix nasa#108, Remove multiple returns from HS_AppInit(), `HS_SbInit…

166f35c

…()` and `HS_TblInit()`

thnkslprpt force-pushed the fix-108-factor-out-multiple-returns branch from 80e6818 to 166f35c Compare

January 14, 2024 08:47

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels