Data field extended to 16 bits

cms-sw · Feb 20, 2020 · 2358a53 · 2358a53
1 parent 7c97972
commit 2358a53
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/DataFormats/FTLDigi/interface/PMTDSimAccumulator.h b/DataFormats/FTLDigi/interface/PMTDSimAccumulator.h
@@ -32,18 +32,18 @@ class PMTDSimAccumulator {
     constexpr static unsigned energyOffset = 4;
     constexpr static unsigned energyMask = 0x3;
     constexpr static unsigned sampleMask = 0xf;
-    constexpr static unsigned dataMask = 0x7fff;
+    constexpr static unsigned dataMask = 0xffff;
 
-    Data() : indices_(0), data_(0) {}
-    Data(unsigned short ei, unsigned short si, unsigned short d) : indices_((ei << energyOffset) | si), data_(d) {}
+    Data() : data_(0), indices_(0) {}
+    Data(unsigned short ei, unsigned short si, unsigned short d) : data_(d), indices_((ei << energyOffset) | si) {}
 
     unsigned int energyIndex() const { return indices_ >> energyOffset; }
     unsigned int sampleIndex() const { return indices_ & sampleMask; }
     unsigned int data() const { return data_ & dataMask; }
 
   private:
-    unsigned char indices_;
     unsigned short data_;
+    unsigned char indices_;
   };
 
   PMTDSimAccumulator() = default;

diff --git a/SimFastTiming/FastTimingCommon/interface/MTDDigitizer.h b/SimFastTiming/FastTimingCommon/interface/MTDDigitizer.h
@@ -66,7 +66,7 @@ namespace mtd_digitizer {
 
     const float minPackChargeLog = minCharge > 0.f ? std::log(minCharge) : -2;
     const float maxPackChargeLog = std::log(maxCharge);
-    constexpr uint16_t base = PMTDSimAccumulator::Data::dataMask + 1;
+    constexpr uint16_t base = PMTDSimAccumulator::Data::dataMask;
 
     simResult.reserve(simData.size());
     // mimicking the digitization
@@ -96,7 +96,7 @@ namespace mtd_digitizer {
                                     const float maxCharge) {
     const float minPackChargeLog = minCharge > 0.f ? std::log(minCharge) : -2;
     const float maxPackChargeLog = std::log(maxCharge);
-    constexpr uint16_t base = PMTDSimAccumulator::Data::dataMask + 1;
+    constexpr uint16_t base = PMTDSimAccumulator::Data::dataMask;
 
     for (const auto& detIdIndexHitInfo : simAccumulator) {
       auto foo = simData.emplace(