Added missing cstdint includes (needed on some systems).
This was required for some reason, for my KUbuntu 24.04 build, using FairSoft jan24 and FairRoot v18.8.2. Probably can't hurt.
Merge request reports
Activity
requested review from @f.uhlig
assigned to @f.uhlig
which version of clang-format did you use? Obviously it is not the one used by the CI since there is an error with the check. According to the CDash page the following patch needs to be applied such that the check works.
diff --git a/core/detectors/trd/CbmTrdParSpadic.h b/core/detectors/trd/CbmTrdParSpadic.h index f574f62aa..6e9343ff0 100644 --- a/core/detectors/trd/CbmTrdParSpadic.h +++ b/core/detectors/trd/CbmTrdParSpadic.h @@ -80,7 +80,7 @@ class CbmTrdParSpadic : public CbmTrdParAsic { static Double_t fgSizeZ; ///< SPADIC half size in z [cm] const std::vector<UInt_t> fVecSpadicChannels = {23, 7, 22, 6, 21, 19, 5, 20, 18, 4, 3, 17, 16, 2, 1, 0, - 31, 30, 29, 15, 14, 28, 27, 13, 11, 26, 12, 10, 25, 9, 24, 8}; + 31, 30, 29, 15, 14, 28, 27, 13, 11, 26, 12, 10, 25, 9, 24, 8}; std::map<UInt_t, UInt_t> fMapAsicChannelToElink = {}; void FillAsicChannelToElinkMap( std::map<UInt_t, UInt_t>* map); ///< Write the eLink to asicChannel mapping to the passed map
it looks like you had the same conclusion.
Edited by Florian Uhligenabled an automatic merge when the pipeline for f3758325 succeeds
Dear @ma.beyer, @n.herrmann, @i.deppner, @f.uhlig, @s.lebedev, @a.bercuci, @p.kaehler, @v.friese, @p.-a.loizeau,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
added BugFix Build System labels