Skip to content

Commit 961d4f5

Browse files
committed
IWYU fixes and cppcheck fixes
1 parent 602e36f commit 961d4f5

3 files changed

Lines changed: 11 additions & 7 deletions

File tree

PWGLF/DataModel/lambdaJetPolarizationIons.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
#include <Framework/AnalysisDataModel.h>
2525

2626
#include <cmath>
27+
#include <cstdint>
2728

2829
namespace o2::aod
2930
{

PWGLF/TableProducer/Strangeness/lambdaJetPolarizationIons.cxx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,10 @@
7272
#include <fastjet/GhostedAreaSpec.hh>
7373
#include <fastjet/JetDefinition.hh>
7474
#include <fastjet/PseudoJet.hh>
75-
#include <sys/types.h>
7675

7776
#include <cmath>
7877
#include <cstddef>
78+
#include <cstdint>
7979
#include <map>
8080
#include <stdexcept>
8181
#include <string>
@@ -303,7 +303,6 @@ struct lambdajetpolarizationions {
303303
// For some binning variables:
304304
int mBinHasRingJet;
305305
int mBinHasRingV0;
306-
std::map<std::string, std::string> metadata;
307306
o2::parameters::GRPMagField* grpmag = nullptr;
308307

309308
// Histogram axes configuration:
@@ -1241,11 +1240,10 @@ struct lambdajetpolarizationions {
12411240
fillEventSelectionQA(++selectionIdx, centrality);
12421241
if (!rctConfigurations.cfgRCTLabel.value.empty() && !rctFlagsChecker(collision))
12431242
return false;
1244-
if (fillHists)
1243+
if (fillHists) {
12451244
fillEventSelectionQA(++selectionIdx, centrality);
12461245

1247-
// Filling histograms previously filled in fillReconstructedEventProperties here, to avoid re-accessing data:
1248-
if (fillHists) {
1246+
// Filling histograms previously filled in fillReconstructedEventProperties here, to avoid re-accessing data:
12491247
histos.fill(HIST("hEventOccupancy"), collisionOccupancy);
12501248
histos.fill(HIST("hCentralityVsOccupancy"), centrality, collisionOccupancy);
12511249
histos.fill(HIST("hInteractionRate"), interactionRate);

PWGLF/Tasks/Strangeness/lambdaJetPolarizationIonsDerived.cxx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@
3636
#include <CommonConstants/MathConstants.h>
3737
#include <CommonConstants/PhysicsConstants.h>
3838
#include <Framework/ASoA.h>
39-
#include <Framework/ASoAHelpers.h>
4039
#include <Framework/AnalysisDataModel.h>
4140
#include <Framework/AnalysisTask.h>
4241
#include <Framework/BinningPolicy.h>
@@ -53,12 +52,18 @@
5352
#include <Math/Vector3Dfwd.h>
5453
#include <Math/Vector4D.h> // IWYU pragma: keep (do not replace with Math/Vector4Dfwd.h)
5554
#include <Math/Vector4Dfwd.h>
55+
#include <TAxis.h>
56+
#include <TH1.h>
57+
#include <TH2.h>
5658
#include <TProfile.h>
59+
#include <TProfile2D.h>
5760
#include <TRandom3.h> // For perpendicular jet direction QAs
5861

5962
#include <algorithm> // std::fill, for resetting the Delta Method accumulators
6063
#include <array>
6164
#include <cmath>
65+
#include <cstddef>
66+
#include <cstdint>
6267
#include <optional>
6368
#include <random>
6469
#include <string>
@@ -1203,7 +1208,7 @@ struct lambdajetpolarizationionsderived {
12031208
/// \note Shared across leadP/leadJet/subJet: the caller resolves which proxy-specific procedure (e.g., LUT for evtMixing) applies.
12041209
// Helper to modify the jet direction for QA and for spurious signal baseline removal tests:
12051210
inline void applyProxyDistortion(ProxyState proxy, float minPtThreshold, ProxyCacheRef cache,
1206-
std::discrete_distribution<int>& etaDist, std::discrete_distribution<int>& phiDist, std::mt19937& rng)
1211+
const std::discrete_distribution<int>& etaDist, const std::discrete_distribution<int>& phiDist, std::mt19937& rng)
12071212
{
12081213
if (!fakePolSwitches.forcePerpToJet && !fakePolSwitches.forceJetDirectionSmudge && !fakePolSwitches.forceRandJet && !fakePolSwitches.forcePreviousJet && !fakePolSwitches.forceDatalikeJet && !fakePolSwitches.doMixedEventProxies) [[likely]] {
12091214
return; // Skip this function if none of the modifications are actually being executed!

0 commit comments

Comments
 (0)