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 #2210, Move variables declared mid-function to the top #2211

Open

thnkslprpt wants to merge 1 commit into nasa:main from thnkslprpt:fix-2210-variables-declared-mid-function

Contributor

thnkslprpt commented Nov 15, 2022 •

edited

Loading

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 Variables declared mid-function #2210
- A couple of stray variables that were declared mid-function were moved to the top of their respective functions.

Testing performed
GitHub CI actions all passing successfully.

Expected behavior changes
All variables declared top of function as per cFS conventions/guidelines.

Contributor Info
Avi Weiss @thnkslprpt

thnkslprpt force-pushed the fix-2210-variables-declared-mid-function branch from 56c3f99 to c54bfd5 Compare

January 25, 2024 22:41


          Fix nasa#2210, Move variables declared mid-function to the top

46df1c0

thnkslprpt force-pushed the fix-2210-variables-declared-mid-function branch from c54bfd5 to 46df1c0 Compare

January 25, 2024 22:44

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

Labels

None yet