Even better - use smart ptr.

cms-sw · Dec 16, 2016 · 4521903 · 4521903
1 parent d4dcae5
commit 4521903
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/L1Trigger/L1TNtuples/plugins/L1GenTreeProducer.cc b/L1Trigger/L1TNtuples/plugins/L1GenTreeProducer.cc
@@ -74,7 +74,7 @@ class L1GenTreeProducer : public edm::EDAnalyzer {
   TTree * tree_;
 
   // data format
-  L1Analysis::L1AnalysisGeneratorDataFormat * l1GenData_;
+  std::unique_ptr<L1Analysis::L1AnalysisGeneratorDataFormat>       l1GenData_;
 
   // EDM input tags
   edm::EDGetTokenT<reco::GenJetCollection> genJetToken_;
@@ -92,7 +92,7 @@ L1GenTreeProducer::L1GenTreeProducer(const edm::ParameterSet& iConfig)
   genParticleToken_ = consumes<reco::GenParticleCollection>(iConfig.getUntrackedParameter<edm::InputTag>("genParticleToken"));
   pileupInfoToken_ = consumes<std::vector<PileupSummaryInfo> >(iConfig.getUntrackedParameter<edm::InputTag>("pileupInfoToken"));
 
-  l1GenData_ = new L1Analysis::L1AnalysisGeneratorDataFormat();
+  l1GenData_ = std::make_unique<L1Analysis::L1AnalysisGeneratorDataFormat>();
 
   // set up output
   tree_=fs_->make<TTree>("L1GenTree", "L1GenTree");
@@ -105,8 +105,6 @@ L1GenTreeProducer::~L1GenTreeProducer()
 
   // do anything here that needs to be done at desctruction time
   // (e.g. close files, deallocate resources etc.)
-
-  delete l1GenData_;
 
 }