Skip to content

Commit

Permalink
fix: Clang-tidy warnings (#1560)
Browse files Browse the repository at this point in the history
#1541 introduced a few clang-tidy warnings that weren't caught. This PR fixes those
  • Loading branch information
paulgessinger committed Sep 28, 2022
1 parent 8fbf7de commit 8bc896e
Showing 1 changed file with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -228,8 +228,9 @@ ActsExamples::ProcessCode ActsExamples::CKFPerformanceWriter::writeT(
// predict if current trajectory is 'duplicate'
bool isDuplicated = m_cfg.duplicatedPredictor(inputFeatures);
// Add to number of duplicated particles
if (isDuplicated)
if (isDuplicated) {
m_nTotalDuplicateTracks++;
}
// Fill the duplication rate
m_duplicationPlotTool.fill(m_duplicationPlotCache, fittedParameters,
isDuplicated);
Expand All @@ -256,8 +257,9 @@ ActsExamples::ProcessCode ActsExamples::CKFPerformanceWriter::writeT(
// 'real' track; others are as 'duplicated'
bool isDuplicated = (itrack != 0);
// the track is associated to the same particle
if (isDuplicated)
if (isDuplicated) {
m_nTotalDuplicateTracks++;
}
// Fill the duplication rate
m_duplicationPlotTool.fill(m_duplicationPlotCache, fittedParameters,
isDuplicated);
Expand All @@ -283,8 +285,9 @@ ActsExamples::ProcessCode ActsExamples::CKFPerformanceWriter::writeT(
m_nTotalMatchedParticles += 1;
// Check if the particle has more than one matched track for the duplicate
// rate
if (nMatchedTracks > 1)
if (nMatchedTracks > 1) {
m_nTotalDuplicateParticles += 1;
}
isReconstructed = true;
}
// Fill efficiency plots
Expand Down

0 comments on commit 8bc896e

Please sign in to comment.