[hist] Rely on injected-class-name

In a class template scope, the injected-class-name allows to omit the
template arguments which makes the code more readable.

Author: hahnjo

hist/histv7/inc/ROOT/RHist.hxx

@@ -91,21 +91,21 @@ public:
    ///
    /// Copying all bin contents can be an expensive operation, depending on the number of bins. If required, users can
    /// explicitly call Clone().
-   RHist(const RHist<BinContentType> &) = delete;
+   RHist(const RHist &) = delete;
    /// Efficiently move construct a histogram.
    ///
    /// After this operation, the moved-from object is invalid.
-   RHist(RHist<BinContentType> &&) = default;
+   RHist(RHist &&) = default;
 
    /// The copy assignment operator is deleted.
    ///
    /// Copying all bin contents can be an expensive operation, depending on the number of bins. If required, users can
    /// explicitly call Clone().
-   RHist<BinContentType> &operator=(const RHist<BinContentType> &) = delete;
+   RHist &operator=(const RHist &) = delete;
    /// Efficiently move a histogram.
    ///
    /// After this operation, the moved-from object is invalid.
-   RHist<BinContentType> &operator=(RHist<BinContentType> &&) = default;
+   RHist &operator=(RHist &&) = default;
 
    ~RHist() = default;
 
@@ -178,7 +178,7 @@ public:
    /// Throws an exception if the axes configurations are not identical.
    ///
    /// \param[in] other another histogram
-   void Add(const RHist<BinContentType> &other)
+   void Add(const RHist &other)
    {
       fEngine.Add(other.fEngine);
       fStats.Add(other.fStats);
@@ -189,7 +189,7 @@ public:
    /// Throws an exception if the axes configurations are not identical.
    ///
    /// \param[in] other another histogram that must not be modified during the operation
-   void AddAtomic(const RHist<BinContentType> &other)
+   void AddAtomic(const RHist &other)
    {
       fEngine.AddAtomic(other.fEngine);
       fStats.AddAtomic(other.fStats);
@@ -207,9 +207,9 @@ public:
    /// Copying all bin contents can be an expensive operation, depending on the number of bins.
    ///
    /// \return the cloned object
-   RHist<BinContentType> Clone() const
+   RHist Clone() const
    {
-      RHist<BinContentType> h(fEngine.Clone());
+      RHist h(fEngine.Clone());
       h.fStats = fStats;
       return h;
    }

hist/histv7/inc/ROOT/RHistConcurrentFiller.hxx

@@ -54,10 +54,10 @@ public:
       }
    }
 
-   RHistConcurrentFiller(const RHistConcurrentFiller<BinContentType> &) = delete;
-   RHistConcurrentFiller(RHistConcurrentFiller<BinContentType> &&) = delete;
-   RHistConcurrentFiller<BinContentType> &operator=(const RHistConcurrentFiller<BinContentType> &) = delete;
-   RHistConcurrentFiller<BinContentType> &operator=(RHistConcurrentFiller<BinContentType> &&) = delete;
+   RHistConcurrentFiller(const RHistConcurrentFiller &) = delete;
+   RHistConcurrentFiller(RHistConcurrentFiller &&) = delete;
+   RHistConcurrentFiller &operator=(const RHistConcurrentFiller &) = delete;
+   RHistConcurrentFiller &operator=(RHistConcurrentFiller &&) = delete;
 
    ~RHistConcurrentFiller()
    {

hist/histv7/inc/ROOT/RHistEngine.hxx

@@ -100,21 +100,21 @@ public:
    ///
    /// Copying all bin contents can be an expensive operation, depending on the number of bins. If required, users can
    /// explicitly call Clone().
-   RHistEngine(const RHistEngine<BinContentType> &) = delete;
+   RHistEngine(const RHistEngine &) = delete;
    /// Efficiently move construct a histogram engine.
    ///
    /// After this operation, the moved-from object is invalid.
-   RHistEngine(RHistEngine<BinContentType> &&) = default;
+   RHistEngine(RHistEngine &&) = default;
 
    /// The copy assignment operator is deleted.
    ///
    /// Copying all bin contents can be an expensive operation, depending on the number of bins. If required, users can
    /// explicitly call Clone().
-   RHistEngine<BinContentType> &operator=(const RHistEngine<BinContentType> &) = delete;
+   RHistEngine &operator=(const RHistEngine &) = delete;
    /// Efficiently move a histogram engine.
    ///
    /// After this operation, the moved-from object is invalid.
-   RHistEngine<BinContentType> &operator=(RHistEngine<BinContentType> &&) = default;
+   RHistEngine &operator=(RHistEngine &&) = default;
 
    ~RHistEngine() = default;
 
@@ -187,7 +187,7 @@ public:
    /// Throws an exception if the axes configurations are not identical.
    ///
    /// \param[in] other another histogram
-   void Add(const RHistEngine<BinContentType> &other)
+   void Add(const RHistEngine &other)
    {
       if (fAxes != other.fAxes) {
          throw std::invalid_argument("axes configurations not identical in Add");
@@ -202,7 +202,7 @@ public:
    /// Throws an exception if the axes configurations are not identical.
    ///
    /// \param[in] other another histogram that must not be modified during the operation
-   void AddAtomic(const RHistEngine<BinContentType> &other)
+   void AddAtomic(const RHistEngine &other)
    {
       if (fAxes != other.fAxes) {
          throw std::invalid_argument("axes configurations not identical in AddAtomic");
@@ -225,9 +225,9 @@ public:
    /// Copying all bin contents can be an expensive operation, depending on the number of bins.
    ///
    /// \return the cloned object
-   RHistEngine<BinContentType> Clone() const
+   RHistEngine Clone() const
    {
-      RHistEngine<BinContentType> h(fAxes.Get());
+      RHistEngine h(fAxes.Get());
       for (std::size_t i = 0; i < fBinContents.size(); i++) {
          h.fBinContents[i] = fBinContents[i];
       }

hist/histv7/inc/ROOT/RHistFillContext.hxx

@@ -40,10 +40,10 @@ private:
 
    /// \sa RHistConcurrentFiller::CreateFillContent()
    explicit RHistFillContext(RHist<BinContentType> &hist) : fHist(&hist), fStats(hist.GetNDimensions()) {}
-   RHistFillContext(const RHistFillContext<BinContentType> &) = delete;
-   RHistFillContext(RHistFillContext<BinContentType> &&) = default;
-   RHistFillContext<BinContentType> &operator=(const RHistFillContext<BinContentType> &) = delete;
-   RHistFillContext<BinContentType> &operator=(RHistFillContext<BinContentType> &&) = default;
+   RHistFillContext(const RHistFillContext &) = delete;
+   RHistFillContext(RHistFillContext &&) = default;
+   RHistFillContext &operator=(const RHistFillContext &) = delete;
+   RHistFillContext &operator=(RHistFillContext &&) = default;
 
 public:
    ~RHistFillContext() { Flush(); }