[RF] Fix invalid proxy and memory leak in RooNonCPEigenDecay

There was an invalid proxy in `RooNonCPEigenDecay`, because it was
initialized via move assignement which is not supported for proxies.

Furthermore, there was a memory leak of the arg that this proxy points
to, which is fixed by making the proxy own that arg.

Author: guitargeek

roofit/roofit/inc/RooNonCPEigenDecay.h

@@ -108,7 +108,7 @@ class RooNonCPEigenDecay : public RooAbsAnaConvPdf {
   RooCategoryProxy _tag;
   RooCategoryProxy _rhoQ;
   RooRealProxy     _correctQ;
-  RooRealProxy     _wQ;
+  RooRealProxy     _wQ; ///< dummy mischarge (must be set to zero!)
   double         _genB0Frac;
   double         _genRhoPlusFrac;
 

roofit/roofit/src/RooNonCPEigenDecay.cxx

@@ -147,13 +147,11 @@ RooNonCPEigenDecay::RooNonCPEigenDecay( const char *name, const char *title,
   _tag      ( "tag",      "CP state",           this, tag      ),
   _rhoQ     ( "rhoQ",     "Charge of the rho",  this, rhoQ     ),
   _correctQ ( "correctQ", "correction of rhoQ", this, correctQ ),
+  _wQ      ( "wQ", "mischarge", this, *(new RooRealVar( "wQ", "wQ", 0 )), true, false, true ),
   _genB0Frac     ( 0 ),
   _genRhoPlusFrac( 0 ),
   _type     ( type )
 {
-  // dummy mischarge (must be set to zero!)
-  _wQ = RooRealProxy( "wQ", "mischarge", this, *(new RooRealVar( "wQ", "wQ", 0 )) );
-
   switch(type) {
   case SingleSided:
     _basisExp = declareBasis( "exp(-@0/@1)",            RooArgList( tau     ) );