Skip to content

Commit 96cabee

Browse files
authored
Merge pull request #47486 from Dr15Jones/remove_adjustToNewProductRegistry
Removed Principal::adjustToNewProductRegistry
2 parents 8f4b703 + 2f3871d commit 96cabee

File tree

5 files changed

+0
-27
lines changed

5 files changed

+0
-27
lines changed

FWCore/Framework/interface/Principal.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,6 @@ namespace edm {
7474

7575
~Principal() override;
7676

77-
bool adjustToNewProductRegistry(ProductRegistry const& reg);
78-
7977
void adjustIndexesAfterProductRegistryAddition();
8078

8179
void fillPrincipal(DelayedReader* reader);

FWCore/Framework/src/EventProcessor.cc

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2034,8 +2034,6 @@ namespace edm {
20342034
void EventProcessor::readAndMergeRun(RunProcessingStatus& iStatus) {
20352035
RunPrincipal& runPrincipal = *iStatus.runPrincipal();
20362036

2037-
bool runOK = runPrincipal.adjustToNewProductRegistry(*preg_);
2038-
assert(runOK);
20392037
runPrincipal.mergeAuxiliary(*input_->runAuxiliary());
20402038
{
20412039
SendSourceTerminationSignalIfException sentry(actReg_.get());
@@ -2063,8 +2061,6 @@ namespace edm {
20632061
input_->processHistoryRegistry().reducedProcessHistoryID(lumiPrincipal.aux().processHistoryID()) ==
20642062
input_->processHistoryRegistry().reducedProcessHistoryID(
20652063
input_->luminosityBlockAuxiliary()->processHistoryID()));
2066-
bool lumiOK = lumiPrincipal.adjustToNewProductRegistry(*preg());
2067-
assert(lumiOK);
20682064
lumiPrincipal.mergeAuxiliary(*input_->luminosityBlockAuxiliary());
20692065
{
20702066
SendSourceTerminationSignalIfException sentry(actReg_.get());

FWCore/Framework/src/Principal.cc

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -149,23 +149,6 @@ namespace edm {
149149
return size;
150150
}
151151

152-
// adjust provenance for input products after new input file has been merged
153-
bool Principal::adjustToNewProductRegistry(ProductRegistry const& reg) {
154-
ProductRegistry::ProductList const& prodsList = reg.productList();
155-
for (auto const& prod : prodsList) {
156-
ProductDescription const& bd = prod.second;
157-
if (!bd.produced() && (bd.branchType() == branchType_)) {
158-
auto cbd = std::make_shared<ProductDescription const>(bd);
159-
auto phb = getExistingProduct(cbd->branchID());
160-
if (phb == nullptr || phb->productDescription().branchName() != cbd->branchName()) {
161-
return false;
162-
}
163-
phb->resetProductDescription(cbd);
164-
}
165-
}
166-
return true;
167-
}
168-
169152
void Principal::addDroppedProduct(ProductDescription const& bd) {
170153
addProductOrThrow(std::make_unique<DroppedDataProductResolver>(std::make_shared<ProductDescription const>(bd)));
171154
}

FWCore/Framework/src/PrincipalCache.cc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@ namespace edm {
4444
for (auto& eventPrincipal : eventPrincipals_) {
4545
if (eventPrincipal) {
4646
eventPrincipal->adjustIndexesAfterProductRegistryAddition();
47-
bool eventOK = eventPrincipal->adjustToNewProductRegistry(*reg);
48-
assert(eventOK);
4947
}
5048
}
5149
}

IOPool/Streamer/src/StreamerInputSource.cc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,8 +299,6 @@ namespace edm::streamer {
299299
void StreamerInputSource::read(EventPrincipal& eventPrincipal) {
300300
if (adjustEventToNewProductRegistry_) {
301301
eventPrincipal.adjustIndexesAfterProductRegistryAddition();
302-
bool eventOK = eventPrincipal.adjustToNewProductRegistry(productRegistry());
303-
assert(eventOK);
304302
adjustEventToNewProductRegistry_ = false;
305303
}
306304
EventSelectionIDVector ids(sendEvent_->eventSelectionIDs());

0 commit comments

Comments
 (0)