Skip to content

Commit 650fde8

Browse files
committed
Improvements based on review
- removed unnecessary includes - removed redundant checks - fixed comments
1 parent b00dec5 commit 650fde8

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

FWCore/Framework/interface/ModuleRegistryUtilities.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ namespace edm {
3939

4040
/** beginStreamFailedForModule holds module id for each module which threw an exception during
4141
* the call to beginStream function. This vector is used to determine which modules should not
42-
* have their endStream called. The vector should be passed to `runEndJobForModules`.
42+
* have their endStream called. The vector should be passed to `runEndStreamForModules`.
4343
* If an exception is thrown, it will be of type cms::Exception.
4444
*/
4545
void runBeginStreamForModules(StreamContext const& iStreamContext,
@@ -53,7 +53,7 @@ namespace edm {
5353
ActivityRegistry& iRegistry,
5454
ExceptionCollector& collector,
5555
std::mutex& collectorMutex,
56-
std::vector<unsigned int> const& beginJobFailedForModule) noexcept;
56+
std::vector<unsigned int> const& beginStreamFailedForModule) noexcept;
5757

5858
} // namespace edm
59-
#endif // FWCore_Framework_ModuleRegistryUtilities_h
59+
#endif // FWCore_Framework_ModuleRegistryUtilities_h

FWCore/Framework/src/GlobalSchedule.cc

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,8 @@ namespace edm {
103103
try {
104104
runBeginJobForModules(gc, modReg, *actReg_, beginJobFailedForModule_);
105105
} catch (cms::Exception& ex) {
106-
if (!exceptionPtr) {
107-
ex.addContext(globalContext);
108-
exceptionPtr = std::current_exception();
109-
}
106+
ex.addContext(globalContext);
107+
exceptionPtr = std::current_exception();
110108
}
111109
}
112110
try {

FWCore/Framework/src/Schedule.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@
3434
#include "FWCore/ParameterSet/interface/ParameterSetDescription.h"
3535
#include "FWCore/ServiceRegistry/interface/ActivityRegistry.h"
3636
#include "FWCore/ServiceRegistry/interface/ModuleConsumesInfo.h"
37-
#include "FWCore/ServiceRegistry/interface/ProcessContext.h"
3837
#include "FWCore/Utilities/interface/Algorithms.h"
3938
#include "FWCore/Utilities/interface/ConvertException.h"
4039
#include "FWCore/Utilities/interface/ExceptionCollector.h"

0 commit comments

Comments
 (0)