diff --git a/core/data/psd/CbmPsdDigi.cxx b/core/data/psd/CbmPsdDigi.cxx index 8fe56886c9593089710d85f55d48a4a1e855652f..3e435f95e38993a4aa9ba2e0ff5f7ee9cd4305b0 100644 --- a/core/data/psd/CbmPsdDigi.cxx +++ b/core/data/psd/CbmPsdDigi.cxx @@ -17,24 +17,6 @@ #include <string> // for basic_string -// --- Default constructor -CbmPsdDigi::CbmPsdDigi() {} - - -// --- Constructor with assignment -CbmPsdDigi::CbmPsdDigi(UInt_t address, Double_t time, Double_t edep) : fuAddress(address), fdTime(time), fdEdep(edep) {} - - -// --- Constructor with module and section number -CbmPsdDigi::CbmPsdDigi(UInt_t moduleId, UInt_t sectionId, Double_t time, Double_t edep) - : fuAddress(0) - , fdTime(time) - , fdEdep(edep) -{ - fuAddress = CbmPsdAddress::GetAddress(moduleId, sectionId); -} - - // --- Copy constructor CbmPsdDigi::CbmPsdDigi(const CbmPsdDigi& other) : fuAddress(other.fuAddress), fdTime(other.fdTime), fdEdep(other.fdEdep) { @@ -45,33 +27,6 @@ CbmPsdDigi::CbmPsdDigi(const CbmPsdDigi& other) : fuAddress(other.fuAddress), fd CbmPsdDigi::CbmPsdDigi(CbmPsdDigi&& other) : fuAddress(other.fuAddress), fdTime(other.fdTime), fdEdep(other.fdEdep) {} -// --- Destructor -CbmPsdDigi::~CbmPsdDigi() {} - - -// --- Assignment operator -CbmPsdDigi& CbmPsdDigi::operator=(const CbmPsdDigi& other) -{ - if (this != &other) { - fuAddress = other.fuAddress; - fdTime = other.fdTime; - fdEdep = other.fdEdep; - } - return *this; -} - - -// --- Move assignment operator -CbmPsdDigi& CbmPsdDigi::operator=(CbmPsdDigi&& other) -{ - if (this != &other) { - fuAddress = other.fuAddress; - fdTime = other.fdTime; - fdEdep = other.fdEdep; - } - return *this; -} - // --- Set address from module and section number void CbmPsdDigi::SetAddress(UInt_t moduleId, UInt_t sectionId) diff --git a/core/data/psd/CbmPsdDigi.h b/core/data/psd/CbmPsdDigi.h index 3c32e8f93243dc9733cd037fc12cf4969f8c4c28..5b6367ab1446f86d697e2bb14913a6cc151da5b3 100644 --- a/core/data/psd/CbmPsdDigi.h +++ b/core/data/psd/CbmPsdDigi.h @@ -36,24 +36,35 @@ class CbmPsdDigi { public: /**@brief Default constructor. **/ - CbmPsdDigi(); + CbmPsdDigi() {} /** @brief Constructor with assignment ** @param address Unique channel address - ** @param edep Energy deposition ** @param time Time [ns] + ** @param edep Energy deposition **/ - CbmPsdDigi(UInt_t address, Double_t time, Double_t edep); + CbmPsdDigi(UInt_t address, Double_t time, Double_t edep) + : fuAddress(address) + , fdTime(time) + , fdEdep(edep) + { + } /** @brief Constructor with detailed assignment. ** @param moduleID Module Identifier ** @param sectionID Section Identifier - ** @param edep Energy deposition ** @param time Time [ns] + ** @param edep Energy deposition **/ - CbmPsdDigi(UInt_t moduleId, UInt_t sectionId, Double_t time, Double_t edep); + CbmPsdDigi(UInt_t moduleId, UInt_t sectionId, Double_t time, Double_t edep) + : fuAddress(0) + , fdTime(time) + , fdEdep(edep) + { + fuAddress = CbmPsdAddress::GetAddress(moduleId, sectionId); + } /** Copy constructor **/ @@ -65,15 +76,15 @@ public: /** Assignment operator **/ - CbmPsdDigi& operator=(const CbmPsdDigi&); + CbmPsdDigi& operator=(const CbmPsdDigi&) = default; /** Move Assignment operator **/ - CbmPsdDigi& operator=(CbmPsdDigi&&); + CbmPsdDigi& operator=(CbmPsdDigi&&) = default; /** Destructor **/ - ~CbmPsdDigi(); + ~CbmPsdDigi() {} /** @brief Class name (static) @@ -139,12 +150,11 @@ public: std::string ToString() const; +private: UInt_t fuAddress = 0; /// Unique channel address Double_t fdTime = -1.; /// Time of measurement [ns] Double_t fdEdep = 0.; /// Energy deposition from FPGA [MeV] - -private: /// BOOST serialization interface friend class boost::serialization::access; template<class Archive> @@ -155,7 +165,7 @@ private: ar& fdEdep; } - ClassDefNV(CbmPsdDigi, 6); + ClassDefNV(CbmPsdDigi, 5); }; #endif // CBMPSDDIGI_H