Skip to content

Commit

Permalink
Use ! instead of not
Browse files Browse the repository at this point in the history
  • Loading branch information
jmcarcell committed Dec 9, 2024
1 parent 6cfa798 commit 4dee9ec
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions k4MarlinWrapper/src/components/MarlinProcessorWrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -131,12 +131,12 @@ std::shared_ptr<marlin::StringParameters> MarlinProcessorWrapper::parseParameter
StatusCode MarlinProcessorWrapper::instantiateProcessor(std::shared_ptr<marlin::StringParameters>& parameters,
Gaudi::Property<std::string>& processorTypeStr) {
auto processorType = marlin::ProcessorMgr::instance()->getProcessor(processorTypeStr);
if (not processorType) {
if (!processorType) {
error() << " Failed to instantiate " << name() << " because processor type could not be determined" << endmsg;
return StatusCode::FAILURE;
}
m_processor = processorType->newProcessor();
if (not m_processor) {
if (!m_processor) {
error() << " Failed to instantiate " << name() << endmsg;
return StatusCode::FAILURE;
}
Expand Down Expand Up @@ -215,7 +215,7 @@ StatusCode MarlinProcessorWrapper::execute(const EventContext&) const {
StatusCode scStatus = eventSvc()->retrieveObject("/Event/LCEventStatus", pStatus);
if (scStatus.isSuccess()) {
bool hasLCEvent = static_cast<LCEventWrapperStatus*>(pStatus)->hasLCEvent;
if (not hasLCEvent) {
if (!hasLCEvent) {
warning() << "LCIO Event reading returned nullptr, so MarlinProcessorWrapper won't execute" << endmsg;
return StatusCode::SUCCESS;
}
Expand Down

0 comments on commit 4dee9ec

Please sign in to comment.