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.

Sign up for GitHub

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels