Skip to content
Snippets Groups Projects

LArCondTPCnv: Fix cppcheck warnings.

Merged Scott Snyder requested to merge ssnyder/athena:cppcheck.LArCondTPCnv-20200506 into master

If we check for an error condition and the check fails, we should't continue.

Here, the checks were for internal logic problems, so change them to assert.

Merge request reports

Pipeline #1613856 failed

Pipeline failed for e8d7509d on ssnyder:cppcheck.LArCondTPCnv-20200506

Approval is optional

Merged by Walter LamplWalter Lampl 4 years ago (May 7, 2020 2:23pm UTC)

Merge details

  • Changes merged into master with 508d9294 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading