From adb8fd7c84883d0f07665a88cd6996cdfd42ca55 Mon Sep 17 00:00:00 2001
From: Chris Jones <jonesc@hep.phy.cam.ac.uk>
Date: Wed, 30 Aug 2023 12:16:53 +0100
Subject: [PATCH] RichDecodedData: make some setters for # hits and PDs private

---
 .../include/RichFutureUtils/RichDecodedData.h | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/Rich/RichFutureUtils/include/RichFutureUtils/RichDecodedData.h b/Rich/RichFutureUtils/include/RichFutureUtils/RichDecodedData.h
index 5ac7ba2f95c..d2eed3c9af2 100644
--- a/Rich/RichFutureUtils/include/RichFutureUtils/RichDecodedData.h
+++ b/Rich/RichFutureUtils/include/RichFutureUtils/RichDecodedData.h
@@ -124,6 +124,16 @@ namespace Rich::Future::DAQ {
     /// Returns the overall total number of RICH hits in the decoded data
     [[nodiscard]] auto nTotalHits() const noexcept { return nTotalHits( Rich::Rich1 ) + nTotalHits( Rich::Rich2 ); }
 
+    /// Returns the total number of active PDs in the decoded data for the given RICH
+    [[nodiscard]] auto nActivePDs( const Rich::DetectorType rich ) const noexcept { return m_nActivePDs[rich]; }
+
+    /// Returns the overall total number of active PDs in the decoded data
+    [[nodiscard]] auto nActivePDs() const noexcept { return nActivePDs( Rich::Rich1 ) + nActivePDs( Rich::Rich2 ); }
+
+    /// Add the given SmartIDs to the decoded data
+    void addSmartIDs( const LHCb::RichSmartID::Vector& ids );
+
+  private:
     /// Append to the number of hits for each RICH
     void addToTotalHits( const DetectorArray<unsigned int>& nHits ) noexcept {
       for ( std::size_t i = 0; i < nHits.size(); ++i ) { m_nTotalHits[i] += nHits[i]; }
@@ -134,12 +144,6 @@ namespace Rich::Future::DAQ {
       m_nTotalHits[rich] += nHits;
     }
 
-    /// Returns the total number of active PDs in the decoded data for the given RICH
-    [[nodiscard]] auto nActivePDs( const Rich::DetectorType rich ) const noexcept { return m_nActivePDs[rich]; }
-
-    /// Returns the overall total number of active PDs in the decoded data
-    [[nodiscard]] auto nActivePDs() const noexcept { return nActivePDs( Rich::Rich1 ) + nActivePDs( Rich::Rich2 ); }
-
     /// Append to the number of active PDs for each RICH
     void addToActivePDs( const DetectorArray<unsigned int>& nPDs ) noexcept {
       for ( std::size_t i = 0; i < nPDs.size(); ++i ) { m_nActivePDs[i] += nPDs[i]; }
@@ -150,9 +154,6 @@ namespace Rich::Future::DAQ {
       m_nActivePDs[rich] += nPDs;
     }
 
-    /// Add the given SmartIDs to the decoded data
-    void addSmartIDs( const LHCb::RichSmartID::Vector& ids );
-
   private:
     /// The total hit count for each RICH detector
     DetectorArray<unsigned int> m_nTotalHits = {{0, 0}};
-- 
GitLab