Dear Giulio,
  I substituted all new cout by message logger and removed commented out 
codes from the code.
  There is a new pull request.
  About other your remarks:

  1. For me, commented out parameters in the example config files in 
/test is a convenient way to document features. Is it really not allowed?
  2. I would like to take some time to study your proposal about using 
c++11 nullptr and  brace-or-equal initializer.

  Please let me know.
                                     regards, Mikhail


On 4/3/14 1:29 AM, Giulio Eulisse wrote:
>
> In GeneratorInterface/Pythia8Interface/plugins/Pythia8Hadronizer.cc:
>
> >    if(fJetMatchingHook) fMasterGen->setUserHooksPtr(fJetMatchingHook);
> >    if(fEmissionVetoHook || fEmissionVetoHook1) {
> > -    cout << "Turning on Emission Veto Hook";
> > -    if(fEmissionVetoHook1) cout << " 1";
> > -    cout << endl;
> > -    int nversion = (int)(1000.*(fMasterGen->settings.parm("Pythia:versionNumber") - 8.));
> > -    if(nversion < 157) {
> > -      cout << "obsolete pythia8 version for this Emission Veto code" << endl;
> > -      cout << "Please update pythia8 version using the instructions here:" << endl;
> > -      cout << "https://twiki.cern.ch/twiki/bin/view/CMS/Pythia8Interface" << endl;
> > -      cout << "or try to use tag V00-01-28 of this interface" << endl;
> > -      throw edm::Exception(edm::errors::Configuration,"Pythia8Interface")
> > -        <<" Obsolete pythia8 version for this Emission Veto code\n";
> > -    }
> > +    std::cout << "Turning on Emission Veto Hook";
> > +    if(fEmissionVetoHook1) std::cout << " 1";
> > +    std::cout << std::endl;
>
> Please use message logger, not std::cout.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/cms-sw/cmssw/pull/2984/files#r11228827>.
>