Skip to content

Commit 72760ec

Browse files
committed
Fix casts
Fixes C-style, redundant, wrong, and dangerous casts (found by Clang-Tidy and cpplint). - Remove redundant casts. (`readability-redundant-casting`) - Remove `const_cast`. (`cppcoreguidelines-pro-type-const-cast`) - Replace C-style casts with `static_cast`. (`google-readability-casting`, cpplint: `readability/casting`) - Replace unsafe C-style casts with `dynamic_cast`. (`cppcoreguidelines-pro-type-cstyle-cast`) - Replace `static_cast` in downcasts with `dynamic_cast`. (`cppcoreguidelines-pro-type-static-cast-downcast`) - Replace `reinterpret_cast` with `static_cast` or `dynamic_cast`. (`cppcoreguidelines-pro-type-reinterpret-cast`) - Replace two-step casting through `void*` with `reinterpret_cast`. (`bugprone-casting-through-void`)
1 parent dc12526 commit 72760ec

13 files changed

Lines changed: 38 additions & 38 deletions

‎DPG/Tasks/AOTEvent/detectorOccupancyQa.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ struct DetectorOccupancyQaTask {
162162
}
163163
// track QA counters
164164
histos.add("nTrackCounter_after_cuts_QA", "", kTH1D, {{12, -0.5, 11.5, "track QA"}});
165-
TAxis* axTrackCounters = reinterpret_cast<TAxis*>(histos.get<TH1>(HIST("nTrackCounter_after_cuts_QA"))->GetXaxis());
165+
TAxis* axTrackCounters = histos.get<TH1>(HIST("nTrackCounter_after_cuts_QA"))->GetXaxis();
166166
axTrackCounters->SetBinLabel(1, "all");
167167
axTrackCounters->SetBinLabel(2, "PVcontrib");
168168
axTrackCounters->SetBinLabel(3, "ptCut");

‎DPG/Tasks/AOTEvent/lumiQa.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,13 +162,13 @@ struct LumiQaTask {
162162
return;
163163
}
164164

165-
hCalibV0A = reinterpret_cast<TH1*>(callst->FindObject("hCalibZeqFV0"));
165+
hCalibV0A = dynamic_cast<TH1*>(callst->FindObject("hCalibZeqFV0"));
166166
if (hCalibV0A == nullptr) {
167167
LOGF(info, "hCalibZeqFV0 histogram is not available for run=%d at timestamp=%llu", runNumber, ts);
168168
return;
169169
}
170170

171-
hCalibT0C = reinterpret_cast<TH1*>(callst->FindObject("hCalibZeqFT0C"));
171+
hCalibT0C = dynamic_cast<TH1*>(callst->FindObject("hCalibZeqFT0C"));
172172
if (hCalibT0C == nullptr) {
173173
LOGF(info, "hCalibZeqFT0C histogram is not available for run=%d at timestamp=%llu", runNumber, ts);
174174
return;

‎DPG/Tasks/AOTEvent/timeDependentQa.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ struct TimeDependentQaTask {
294294
// ### QA event selection bits
295295
int nEvSelBits = enNumEvSelBits;
296296
histos.add("hSecondsEventSelBits", "", kTH2F, {axisSecondsWideBins, {nEvSelBits, -0.5, nEvSelBits - 0.5, "Monitoring of event selection bits"}});
297-
TAxis* axSelBits = reinterpret_cast<TAxis*>(histos.get<TH2>(HIST("hSecondsEventSelBits"))->GetYaxis());
297+
TAxis* axSelBits = histos.get<TH2>(HIST("hSecondsEventSelBits"))->GetYaxis();
298298
axSelBits->SetBinLabel(1 + enCollisionsAll, "collisionsAll");
299299
axSelBits->SetBinLabel(1 + enIsTriggerTVX, "IsTriggerTVX");
300300
axSelBits->SetBinLabel(1 + enNoTimeFrameBorder, "NoTimeFrameBorder");
@@ -330,7 +330,7 @@ struct TimeDependentQaTask {
330330
// ### QA RCT flags
331331
int nRctFlagsTotal = enNumRctFlagsTotal;
332332
histos.add("hSecondsRCTflags", "", kTH2F, {axisSecondsWideBins, {nRctFlagsTotal + 2, -0.5, nRctFlagsTotal + 2 - 0.5, "Monitoring of RCT flags"}});
333-
axRctFlags = reinterpret_cast<TAxis*>(histos.get<TH2>(HIST("hSecondsRCTflags"))->GetYaxis());
333+
axRctFlags = histos.get<TH2>(HIST("hSecondsRCTflags"))->GetYaxis();
334334
axRctFlags->SetBinLabel(1, "NcollisionsSel8");
335335
axRctFlags->SetBinLabel(2, "CcdbNotFound");
336336
axRctFlags->SetBinLabel(3 + kCPVBad, "CPVBad");

‎DPG/Tasks/AOTTrack/MonitorFilterBit.cxx‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -103,15 +103,15 @@ struct CheckFilterBit {
103103
using Tracksextension = soa::Join<aod::Tracks, aod::TracksExtra, aod::TrackSelection, aod::TrackSelectionExtension, aod::TracksDCA>;
104104
using TracksextensionMC = soa::Join<Tracksextension, aod::McTrackLabels>;
105105
SliceCache cache;
106-
Partition<Tracksextension> positiveTPConlyTracks = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcNClsFindable > (uint8_t)0 && o2::aod::track::itsChi2NCl < (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
107-
Partition<Tracksextension> negativeTPConlyTracks = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcNClsFindable > (uint8_t)0 && o2::aod::track::itsChi2NCl < (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
108-
Partition<Tracksextension> positiveITSonlyTracks = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcChi2NCl<(float_t)0 && o2::aod::track::itsChi2NCl>(float_t) 0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
109-
Partition<Tracksextension> negativeITSonlyTracks = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcChi2NCl < (float_t)0 && o2::aod::track::itsChi2NCl > (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 &&(o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
110-
111-
Partition<TracksextensionMC> positiveTPConlyTracksMC = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcNClsFindable > (uint8_t)0 && o2::aod::track::itsChi2NCl < (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
112-
Partition<TracksextensionMC> negativeTPConlyTracksMC = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcNClsFindable > (uint8_t)0 && o2::aod::track::itsChi2NCl < (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
113-
Partition<TracksextensionMC> positiveITSonlyTracksMC = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcChi2NCl<(float_t)0 && o2::aod::track::itsChi2NCl>(float_t) 0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
114-
Partition<TracksextensionMC> negativeITSonlyTracksMC = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcChi2NCl < (float_t)0 && o2::aod::track::itsChi2NCl > (float_t)0 && o2::aod::track::trackType == (uint8_t)o2::aod::track::TrackTypeEnum::Track; // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 &&(o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
106+
Partition<Tracksextension> positiveTPConlyTracks = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcNClsFindable > static_cast<uint8_t>(0) && o2::aod::track::itsChi2NCl < static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
107+
Partition<Tracksextension> negativeTPConlyTracks = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcNClsFindable > static_cast<uint8_t>(0) && o2::aod::track::itsChi2NCl < static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
108+
Partition<Tracksextension> positiveITSonlyTracks = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcChi2NCl<static_cast<float_t>(0) && o2::aod::track::itsChi2NCl> static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
109+
Partition<Tracksextension> negativeITSonlyTracks = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcChi2NCl < static_cast<float_t>(0) && o2::aod::track::itsChi2NCl > static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 &&(o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
110+
111+
Partition<TracksextensionMC> positiveTPConlyTracksMC = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcNClsFindable > static_cast<uint8_t>(0) && o2::aod::track::itsChi2NCl < static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
112+
Partition<TracksextensionMC> negativeTPConlyTracksMC = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcNClsFindable > static_cast<uint8_t>(0) && o2::aod::track::itsChi2NCl < static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); //&& (o2::aod::track::detectorMap & o2::aod::track::TPC) ==o2::aod::track::TPC && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==0;
113+
Partition<TracksextensionMC> positiveITSonlyTracksMC = o2::aod::track::signed1Pt > fzero&& o2::aod::track::tpcChi2NCl<static_cast<float_t>(0) && o2::aod::track::itsChi2NCl> static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 && (o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
114+
Partition<TracksextensionMC> negativeITSonlyTracksMC = o2::aod::track::signed1Pt < fzero && o2::aod::track::tpcChi2NCl < static_cast<float_t>(0) && o2::aod::track::itsChi2NCl > static_cast<float_t>(0) && o2::aod::track::trackType == static_cast<uint8_t>(o2::aod::track::TrackTypeEnum::Track); // && (o2::aod::track::detectorMap & o2::aod::track::TPC) ==0 &&(o2::aod::track::detectorMap & o2::aod::track::ITS) ==o2::aod::track::ITS && o2::aod::track::passedITSNCls==true;// && o2::aod::track::itsNCls==7;
115115

116116
void init(InitContext const&)
117117
{

‎DPG/Tasks/AOTTrack/PID/TOF/qaPIDTOFEvTime.cxx‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -436,11 +436,11 @@ struct tofPidCollisionTimeQa {
436436
histos.fill(HIST("goodforevtime/tofSignalPerCollision"), ncolls % 6000, trk.tofSignal());
437437
}
438438
}
439-
static_cast<TEfficiency*>(listEfficiency->FindObject("effTOFEvTime"))->Fill(t.isEvTimeTOF(), nTracksWithTOF);
440-
static_cast<TEfficiency*>(listEfficiency->FindObject("effT0AEvTime"))->Fill(collision.has_foundFT0() && collision.t0ACorrectedValid(), nTracksWithTOF);
441-
static_cast<TEfficiency*>(listEfficiency->FindObject("effT0CEvTime"))->Fill(collision.has_foundFT0() && collision.t0CCorrectedValid(), nTracksWithTOF);
442-
static_cast<TEfficiency*>(listEfficiency->FindObject("effT0ACEvTime"))->Fill(collision.has_foundFT0() && collision.t0ACValid(), nTracksWithTOF);
443-
static_cast<TEfficiency*>(listEfficiency->FindObject("effTOFT0ACEvTime"))->Fill(t.isEvTimeTOF() && collision.has_foundFT0() && collision.t0ACorrectedValid(), nTracksWithTOF);
439+
dynamic_cast<TEfficiency*>(listEfficiency->FindObject("effTOFEvTime"))->Fill(t.isEvTimeTOF(), nTracksWithTOF);
440+
dynamic_cast<TEfficiency*>(listEfficiency->FindObject("effT0AEvTime"))->Fill(collision.has_foundFT0() && collision.t0ACorrectedValid(), nTracksWithTOF);
441+
dynamic_cast<TEfficiency*>(listEfficiency->FindObject("effT0CEvTime"))->Fill(collision.has_foundFT0() && collision.t0CCorrectedValid(), nTracksWithTOF);
442+
dynamic_cast<TEfficiency*>(listEfficiency->FindObject("effT0ACEvTime"))->Fill(collision.has_foundFT0() && collision.t0ACValid(), nTracksWithTOF);
443+
dynamic_cast<TEfficiency*>(listEfficiency->FindObject("effTOFT0ACEvTime"))->Fill(t.isEvTimeTOF() && collision.has_foundFT0() && collision.t0ACorrectedValid(), nTracksWithTOF);
444444
}
445445
}
446446
PROCESS_SWITCH(tofPidCollisionTimeQa, processData, "Process data", true);

‎DPG/Tasks/AOTTrack/V0Cascades/qaEfficiencyV0s.cxx‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -125,12 +125,12 @@ struct QaEfficiencyV0s {
125125
}
126126

127127
for (int i = 0; i < nSpecies; i++) {
128-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies))->SetPassedHistogram(*histograms[i][kHistoPtNum].get(), "f");
129-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies))->SetTotalHistogram(*histograms[i][kHistoPtDen].get(), "f");
130-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 1))->SetPassedHistogram(*histogramsPrm[i][kHistoPtNum].get(), "f");
131-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 1))->SetTotalHistogram(*histogramsPrm[i][kHistoPtDen].get(), "f");
132-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 2))->SetPassedHistogram(*histogramsPrmRap[i][kHistoPtNum].get(), "f");
133-
static_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 2))->SetTotalHistogram(*histogramsPrmRap[i][kHistoPtDen].get(), "f");
128+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies))->SetPassedHistogram(*histograms[i][kHistoPtNum].get(), "f");
129+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies))->SetTotalHistogram(*histograms[i][kHistoPtDen].get(), "f");
130+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 1))->SetPassedHistogram(*histogramsPrm[i][kHistoPtNum].get(), "f");
131+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 1))->SetTotalHistogram(*histogramsPrm[i][kHistoPtDen].get(), "f");
132+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 2))->SetPassedHistogram(*histogramsPrmRap[i][kHistoPtNum].get(), "f");
133+
dynamic_cast<TEfficiency*>(listEfficiencyMC->At(i * nSpecies + 2))->SetTotalHistogram(*histogramsPrmRap[i][kHistoPtDen].get(), "f");
134134
}
135135
}
136136
};

‎DPG/Tasks/AOTTrack/derivedDataCreatorD0Calibration.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ struct DerivedDataCreatorD0Calibration {
152152
ccdb->setLocalObjectValidityChecking();
153153

154154
if (cfgDownsampling.apply) {
155-
histDownSampl.setObject(reinterpret_cast<TH1D*>(ccdb->getSpecific<TH1D>(cfgDownsampling.pathCcdbWeights)));
155+
histDownSampl.setObject(static_cast<TH1D*>(ccdb->getSpecific<TH1D>(cfgDownsampling.pathCcdbWeights)));
156156
}
157157

158158
if (cfgMl.apply) {
@@ -537,7 +537,7 @@ struct DerivedDataCreatorD0Calibration {
537537
collision.covYZ(),
538538
collision.covZZ(),
539539
collision.numContrib(),
540-
uint8_t(std::round(collision.centFT0C())),
540+
static_cast<uint8_t>(std::round(collision.centFT0C())),
541541
getCompressedOccupancy(collision.trackOccupancyInTimeRange()),
542542
getCompressedOccupancy(collision.ft0cOccupancyInTimeRange()),
543543
eventIR.orbit,

‎DPG/Tasks/AOTTrack/qaEfficiency.cxx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1359,15 +1359,15 @@ struct QaEfficiency {
13591359

13601360
const char* partName = particleName(pdgSign, id);
13611361
LOG(debug) << "Filling efficiency for particle " << static_cast<int>(id) << " " << partName;
1362-
THashList* subList = static_cast<THashList*>(listEfficiencyMC->FindObject(partName));
1362+
THashList* subList = dynamic_cast<THashList*>(listEfficiencyMC->FindObject(partName));
13631363
if (!subList) {
13641364
LOG(warning) << "Cannot find list of efficiency objects for particle " << partName;
13651365
return;
13661366
}
13671367

13681368
// Filling 1D efficiencies
13691369
auto doFillEfficiency = [&](const TString& effname, const auto& num, const auto& den) {
1370-
TEfficiency* eff = static_cast<TEfficiency*>(subList->FindObject(effname));
1370+
TEfficiency* eff = dynamic_cast<TEfficiency*>(subList->FindObject(effname));
13711371
if (!eff) {
13721372
LOG(warning) << "Cannot find TEfficiency " << effname;
13731373
return;
@@ -1437,7 +1437,7 @@ struct QaEfficiency {
14371437

14381438
// Filling 2D efficiencies
14391439
auto fillEfficiency2D = [&](const TString& effname, const auto& num, const auto& den) {
1440-
TEfficiency* eff = static_cast<TEfficiency*>(subList->FindObject(effname));
1440+
TEfficiency* eff = dynamic_cast<TEfficiency*>(subList->FindObject(effname));
14411441
if (!eff) {
14421442
LOG(warning) << "Cannot find TEfficiency " << effname;
14431443
return;

‎DPG/Tasks/AOTTrack/qaEventTrackLite.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -336,8 +336,8 @@ struct qaEventTrackLite {
336336
Filter itsChi2 = (bTpcOnly.node() == true) || (chi2ItsMin < o2::aod::track::itsChi2NCl && o2::aod::track::itsChi2NCl < chi2ItsMax);
337337
// TPC
338338
Filter tpcChi2s = (bItsStandalone.node() == true) || (chi2TpcMin < o2::aod::track::tpcChi2NCl && o2::aod::track::tpcChi2NCl < chi2TpcMax);
339-
Filter tpcNclusters = (bItsStandalone.node() == true) || (o2::aod::dpgtrack::tpcNClsFound > (int16_t)nClusterTpcMin);
340-
Filter tpcNcrossedRows = (bItsStandalone.node() == true) || (o2::aod::dpgtrack::tpcNClsCrossedRows > (int16_t)nCrossedRowsTpcMin);
339+
Filter tpcNclusters = (bItsStandalone.node() == true) || (o2::aod::dpgtrack::tpcNClsFound > static_cast<int16_t>(nClusterTpcMin));
340+
Filter tpcNcrossedRows = (bItsStandalone.node() == true) || (o2::aod::dpgtrack::tpcNClsCrossedRows > static_cast<int16_t>(nCrossedRowsTpcMin));
341341
Filter tpcNcrossedRowsOverFindableClusters = (bItsStandalone.node() == true) || (o2::aod::dpgtrack::tpcCrossedRowsOverFindableCls > nCrossedRowsTpcOverFindableClustersTpcMin);
342342
// TOF
343343
Filter tofChi = o2::aod::track::tofChi2 > chi2TofMin;

‎DPG/Tasks/AOTTrack/qaMatchEff.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1565,7 +1565,7 @@ struct qaMatchEff {
15651565
//
15661566
// here n of clusters of TPC assigned to float for histos (and to hack it if needed) :)
15671567
//
1568-
Float_t clustpc = (Float_t)track.tpcNClsFound();
1568+
Float_t clustpc = static_cast<Float_t>(track.tpcNClsFound());
15691569
// Float_t findcltpc = (Float_t)track.tpcNClsFindable();
15701570
// Float_t crowstpc = (Float_t)track.tpcNClsCrossedRows();
15711571
// Float_t finclusmincrotpc = (Float_t)track.tpcNClsFindableMinusCrossedRows();

0 commit comments

Comments
 (0)