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 #24, Resolve strict cppcheck style warnings #26

Merged
merged 1 commit into from
May 11, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented May 9, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
CI

Expected behavior changes
None, just squashes strict cppcheck style warnings

System(s) tested on
CI

Additional context
None

Third party code
None

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

@skliper skliper added this to the Draco milestone May 9, 2022
@skliper skliper force-pushed the fix24-strict_cppcheck_warnings branch from 5d5a2f6 to 2b79243 Compare May 9, 2022 21:32
@astrogeco
Copy link
Contributor

CCB:2022-05-11 APPROVED

  • Using BufPtr is more appropriate

@astrogeco astrogeco merged commit a75b4f4 into nasa:main May 11, 2022
@skliper skliper deleted the fix24-strict_cppcheck_warnings branch June 9, 2022 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strict cppcheck static analysis style warnings
3 participants