Hi Slava,

I tried to comply with your requests, but I didn't understand the one that
says 'why not with this from the beginning' Could you explain that one?

Thanks,
-Clint


On Tue, Apr 8, 2014 at 11:50 AM, Slava Krutelyov
<notifications@github.com>wrote:

> In RecoJets/JetProducers/plugins/BasicToPFJet.h:
>
> > +#define BasicToPFJet_h
> > +
> > +// system include files
> > +#include <memory>
> > +#include <vector>
> > +#include <sstream>
> > +
> > +// user include files
> > +#include "FWCore/PluginManager/interface/ModuleDef.h"
> > +#include "FWCore/Framework/interface/MakerMacros.h"
> > +#include "FWCore/Framework/interface/Frameworkfwd.h"
> > +#include "FWCore/Framework/interface/EDProducer.h"
> > +#include "FWCore/ParameterSet/interface/ParameterSet.h"
> > +#include "DataFormats/JetReco/
> interface/CATopJetTagInfo.h"
> > +#include "DataFormats/JetReco/interface/BasicJet.h"
> > +#include "DataFormats/JetReco/interface/CaloJet.h"
>
> Looking at this, I think the following are not needed
>
>    - CaloJet.h
>    - CompositeCandidate.h
>    - AddFourMomenta.h
>    - CandMatchMap.h
>    - TriggerTypeDefs.h
>    - HLTFilter.h
>
> please cleanup
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/cms-sw/cmssw/pull/3220/files#r11396210>
> .
>