Skip to content

Commit 8a1a887

Browse files
PWGDQ: fix MEGALinter errors - replace std::string parameters - remove unused members
1 parent 3228317 commit 8a1a887

1 file changed

Lines changed: 3 additions & 10 deletions

File tree

PWGDQ/Tasks/qaMatching.cxx

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,6 @@ struct QaMatching {
255255

256256
static constexpr int GlobalTrackTypeMax = 2;
257257
static constexpr int MchMidTrackType = 3;
258-
static constexpr int FirstDecayMotherRank = 2;
259258
static constexpr int MftTrackTypeStandard = 0;
260259
static constexpr int MftTrackTypeCA = 1;
261260
static constexpr int ThetaAbsBoundaryDeg = 3;
@@ -531,9 +530,6 @@ struct QaMatching {
531530

532531
std::unordered_map<int64_t, int32_t> mftTrackCovs;
533532

534-
std::vector<std::pair<int64_t, int64_t>> fMatchablePairs;
535-
MatchingCandidates fMatchingCandidates;
536-
std::vector<int64_t> fTaggedMuons;
537533

538534
using MuonPair = std::pair<std::pair<int64_t, uint64_t>, std::pair<int64_t, uint64_t>>;
539535
using GlobalMuonPair = std::pair<std::pair<int64_t, std::vector<MatchingCandidate>>, std::pair<int64_t, std::vector<MatchingCandidate>>>;
@@ -562,8 +558,6 @@ struct QaMatching {
562558
&registryMatching9}};
563559
HistogramRegistry registryDimuon{"registryDimuon", {}};
564560

565-
std::unordered_map<std::string, o2::framework::HistPtr> matchingHistos;
566-
Matrix<o2::framework::HistPtr, 4, 4> dimuonHistos;
567561

568562
Produces<o2::aod::QaMatchingEvents> qaMatchingEvents;
569563
Produces<o2::aod::QaMatchingMCHTrack> qaMatchingMCHTrack;
@@ -587,7 +581,7 @@ struct QaMatching {
587581
o2::framework::HistPtr etaPdgNum;
588582
o2::framework::HistPtr etaPdgDen;
589583

590-
EfficiencyPlotter(std::string path, std::string title,
584+
EfficiencyPlotter(const std::string& path, const std::string& title,
591585
HistogramRegistry& registry, bool createPdgMomHistograms)
592586
{
593587
AxisSpec pAxis = {100, 0, 100, "p (GeV/c)"};
@@ -845,7 +839,7 @@ struct QaMatching {
845839

846840
HistogramRegistry* registry;
847841

848-
MatchingPlotter(std::string path,
842+
MatchingPlotter(const std::string& path,
849843
HistogramRegistry* reg,
850844
bool createPdgMomHistograms,
851845
int mftMultMax,
@@ -975,7 +969,6 @@ struct QaMatching {
975969
}
976970
};
977971
std::unique_ptr<MatchingPlotter> fChi2MatchingPlotter;
978-
std::map<std::string, std::unique_ptr<HistogramRegistry>> fMatchingHistogramRegistries;
979972
std::map<std::string, std::unique_ptr<MatchingPlotter>> fMatchingPlotters;
980973
std::unique_ptr<MatchingPlotter> fTaggedMuonsMatchingPlotter;
981974

@@ -2761,7 +2754,7 @@ struct QaMatching {
27612754
TMUON const& muonTracks,
27622755
TMFT const& mftTracks,
27632756
CMFT const& mftCovs,
2764-
std::string funcName,
2757+
const std::string& funcName,
27652758
float matchingPlaneZ,
27662759
int extrapMethod,
27672760
const std::vector<std::pair<int64_t, int64_t>>& matchablePairs,

0 commit comments

Comments
 (0)