[RF] Implement move constructor for RooAbsCollection and RooArgSet

Author: guitargeek

roofit/roofitcore/inc/RooAbsCollection.h

@@ -52,6 +52,9 @@ class RooAbsCollection : public TObject, public RooPrintable {
   RooAbsCollection& assignValueOnly(const RooAbsCollection& other, Bool_t oneSafe=kFALSE) ;
   void assignFast(const RooAbsCollection& other, Bool_t setValDirty=kTRUE) ;
 
+  // Move constructor
+  RooAbsCollection(RooAbsCollection && other);
+
   // Copy list and contents (and optionally 'deep' servers)
   RooAbsCollection *snapshot(Bool_t deepCopy=kTRUE) const ;
   Bool_t snapshot(RooAbsCollection& output, Bool_t deepCopy=kTRUE) const ;

roofit/roofitcore/inc/RooArgList.h

@@ -61,6 +61,8 @@ class RooArgList : public RooAbsCollection {
   // to a copied list. The variables in the copied list are independent
   // of the original variables.
   RooArgList(const RooArgList& other, const char *name="");
+  /// Move constructor.
+  RooArgList(RooArgList && other) : RooAbsCollection(std::move(other)) {}
   virtual TObject* clone(const char* newname) const { return new RooArgList(*this,newname); }
   virtual TObject* create(const char* newname) const { return new RooArgList(newname); }
   RooArgList& operator=(const RooArgList& other) { RooAbsCollection::operator=(other) ; return *this ; }

roofit/roofitcore/inc/RooArgSet.h

@@ -74,6 +74,8 @@ class RooArgSet : public RooAbsCollection {
   }
 
   RooArgSet(const RooArgSet& other, const char *name="");
+  /// Move constructor.
+  RooArgSet(RooArgSet && other) : RooAbsCollection(std::move(other)) {}
 
   RooArgSet(const RooArgSet& set1, const RooArgSet& set2,
             const char *name="");

roofit/roofitcore/src/RooAbsCollection.cxx

@@ -119,6 +119,20 @@ RooAbsCollection::RooAbsCollection(const RooAbsCollection& other, const char *na
 }
 
 
+////////////////////////////////////////////////////////////////////////////////
+/// Move constructor.
+
+RooAbsCollection::RooAbsCollection(RooAbsCollection&& other) :
+  TObject(other),
+  RooPrintable(other),
+  _list(std::move(other._list)),
+  _ownCont(other._ownCont),
+  _name(std::move(other._name)),
+  _allRRV(other._allRRV),
+  _sizeThresholdForMapSearch(other._sizeThresholdForMapSearch)
+{
+}
+
 
 ////////////////////////////////////////////////////////////////////////////////
 /// Destructor