From 3fe58d64f18a629a0b95958d75f22af3236d8c72 Mon Sep 17 00:00:00 2001 From: Christopher Jones Date: Thu, 26 Sep 2013 17:17:15 -0500 Subject: [PATCH] Backport fix for EDAlias and failing edm::Refs --- FWCore/Framework/interface/OutputModule.h | 5 ----- FWCore/Framework/src/OutputModule.cc | 7 ------- 2 files changed, 12 deletions(-) diff --git a/FWCore/Framework/interface/OutputModule.h b/FWCore/Framework/interface/OutputModule.h index 0b73990d02b9a..29ac1df2d74af 100644 --- a/FWCore/Framework/interface/OutputModule.h +++ b/FWCore/Framework/interface/OutputModule.h @@ -113,10 +113,6 @@ namespace edm { return droppedBranchIDToKeptBranchID_; } - std::map const& keptBranchIDToDroppedBranchID() { - return keptBranchIDToDroppedBranchID_; - } - private: int maxEvents_; @@ -161,7 +157,6 @@ namespace edm { // needed because of possible EDAliases. // filled in only if key and value are different. std::map droppedBranchIDToKeptBranchID_; - std::map keptBranchIDToDroppedBranchID_; std::unique_ptr branchIDLists_; BranchIDLists const* origBranchIDLists_; diff --git a/FWCore/Framework/src/OutputModule.cc b/FWCore/Framework/src/OutputModule.cc index 04e0a3a4bef94..ba3f5f426c3e6 100644 --- a/FWCore/Framework/src/OutputModule.cc +++ b/FWCore/Framework/src/OutputModule.cc @@ -117,7 +117,6 @@ namespace edm { selectors_(), selector_config_id_(), droppedBranchIDToKeptBranchID_(), - keptBranchIDToDroppedBranchID_(), branchIDLists_(new BranchIDLists), origBranchIDLists_(nullptr), branchParents_(), @@ -249,7 +248,6 @@ namespace edm { if(keptBranchID != branchID) { // An EDAlias branch was persisted. droppedBranchIDToKeptBranchID_.insert(std::make_pair(branchID.id(), keptBranchID.id())); - keptBranchIDToDroppedBranchID_.insert(std::make_pair(keptBranchID.id(), branchID.id())); } } } @@ -454,11 +452,6 @@ namespace edm { // Check for branches dropped while an EDAlias was kept. for(BranchIDList& branchIDList : *branchIDLists_) { for(BranchID::value_type& branchID : branchIDList) { - // Replace BranchID of each kept alias branch with zero, so only the product ID of the original branch will be accessible. - std::map::const_iterator kiter = keptBranchIDToDroppedBranchID_.find(branchID); - if(kiter != keptBranchIDToDroppedBranchID_.end()) { - branchID = 0; - } // Replace BranchID of each dropped branch with that of the kept alias, so the alias branch will have the product ID of the original branch. std::map::const_iterator iter = droppedBranchIDToKeptBranchID_.find(branchID); if(iter != droppedBranchIDToKeptBranchID_.end()) {