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

WIP: Fix compiler warnings #807

Open

pgrete wants to merge 4 commits into develop from pgrete/fix-compiler-warnings

Collaborator

pgrete commented Jan 2, 2023

PR Summary

PR Checklist

Code passes cpplint
New features are documented.
Adds a test for any bugs fixed. Adds tests for new features.
Code is formatted
Changes are summarized in CHANGELOG.md
CI has been triggered on Darwin for performance regression tests.
(@lanl.gov employees) Update copyright on changed files

pgrete added 4 commits

January 2, 2023 15:11


          WIP: fix compiler warnings

743e72d


          WIP: Remove unused (?) check interval

46549bd


          WIP: remove unused params

1bc2a2f


          WIP: more unused vars and undefined comparisons

4f10fe5

Collaborator

Yurlungur commented Apr 14, 2023

@pgrete what's the status of this PR/branch?

Collaborator Author

pgrete commented Apr 21, 2023

I'm still working on it, so it's legit WIP

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