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 cppcheck #1275

Merged
merged 6 commits into from Sep 8, 2021
Merged

Conversation

pinkenburg
Copy link
Contributor

Types of changes

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work for users)
  • Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

cppcheck found a bad array initialization in TrkrClusterv1.cc which probably never affected the code. Fixed a few other benign cppcheck performance warnings

TODOs (if applicable)

Links to other PRs in macros and calibration repositories (if applicable)

@sphenix-jenkins-ci sphenix-jenkins-ci bot added CI-cpp-check-PENDING cpp-check is still running. CI-build-gcc-8.3-clang-PENDING build-gcc-8.3-clang is still running. CI-build-gcc-8.3-new-PENDING build-gcc-8.3-new is still running. CI-build-gcc-8.3-scan-PENDING build-gcc-8.3-scan is still running. CI-DST-readback-PENDING DST-readback is still running. CI-valgrind-PENDING valgrind is still running. CI-calo-QA-PENDING calo-QA is still running. CI-track-low-occ-QA-PENDING track-low-occ-QA is still running. CI-track-high-occ-QA-PENDING track-high-occ-QA is still running. CI-DST-readback-PASS DST-readback passed. CI-build-gcc-8.3-clang-PASS build-gcc-8.3-clang passed. CI-build-gcc-8.3-scan-PASS build-gcc-8.3-scan passed. CI-cpp-check-AVAILABLE cpp-check result is available. CI-calo-QA-AVAILABLE calo-QA result is available. CI-track-low-occ-QA-AVAILABLE track-low-occ-QA result is available. CI-track-high-occ-QA-AVAILABLE track-high-occ-QA result is available. and removed CI-DST-readback-PENDING DST-readback is still running. CI-build-gcc-8.3-clang-PENDING build-gcc-8.3-clang is still running. CI-build-gcc-8.3-scan-PENDING build-gcc-8.3-scan is still running. CI-cpp-check-PENDING cpp-check is still running. CI-calo-QA-PENDING calo-QA is still running. CI-track-low-occ-QA-PENDING track-low-occ-QA is still running. CI-track-high-occ-QA-PENDING track-high-occ-QA is still running. CI-valgrind-PENDING valgrind is still running. labels Sep 8, 2021
@sphenix-jenkins-ci sphenix-jenkins-ci bot added CI-valgrind-AVAILABLE valgrind result is available. CI-build-gcc-8.3-new-PASS build-gcc-8.3-new passed. and removed CI-build-gcc-8.3-new-PENDING build-gcc-8.3-new is still running. labels Sep 8, 2021
@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit a5e4b2146c2aa45c2aa55a7bad588f5fc58588d4:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@pinkenburg pinkenburg merged commit d0721a8 into sPHENIX-Collaboration:master Sep 8, 2021
@pinkenburg pinkenburg deleted the fix-cppcheck branch September 8, 2021 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-build-gcc-8.3-clang-PASS build-gcc-8.3-clang passed. CI-build-gcc-8.3-new-PASS build-gcc-8.3-new passed. CI-build-gcc-8.3-scan-PASS build-gcc-8.3-scan passed. CI-calo-QA-AVAILABLE calo-QA result is available. CI-cpp-check-AVAILABLE cpp-check result is available. CI-DST-readback-PASS DST-readback passed. CI-track-high-occ-QA-AVAILABLE track-high-occ-QA result is available. CI-track-low-occ-QA-AVAILABLE track-low-occ-QA result is available. CI-valgrind-AVAILABLE valgrind result is available.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant