Skip to content
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 #218, #56 - Resolve static analysis warnings #226

Merged
merged 5 commits into from
Apr 18, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Apr 8, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
Build/unit test and run static analysis

Expected behavior changes
None, just cleaned up about half the static analysis warnings

System(s) tested on

  • Hardware: i5/wsl
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit

Additional context
Note goto removal issue is still open which would address another significant set of warnings (#225)

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Apr 8, 2022
@skliper skliper added this to the Draco milestone Apr 8, 2022
@astrogeco astrogeco added CCB:Approved and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Apr 13, 2022
@astrogeco
Copy link
Contributor

CCB:2022-04-13 APPROVED

@astrogeco astrogeco merged commit 869e9d4 into nasa:main Apr 18, 2022
@skliper skliper deleted the fix218-static_analysis branch June 10, 2022 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants