simplifying the logic

Author: ehellbar

Framework/Core/src/runDataProcessing.cxx

@@ -2957,13 +2957,10 @@ int doMain(int argc, char** argv, o2::framework::WorkflowSpec const& workflow,
     // once from the actual definition in the child, a second time from the
     // configuration they get passed by their parents.
     for (auto& dp : importedWorkflow) {
-      auto found = std::find_if(physicalWorkflow.begin(), physicalWorkflow.end(),
-                                [&name = dp.name](DataProcessorSpec const& spec) { return spec.name == name; });
-      // also checking the workflow for processors with the same name but from a different executable,
-      // adding them to the workflow to trigger the check for duplicate names in the MATERIALISE_WORKFLOW state
-      auto duplicate_name = std::find_if(dataProcessorInfos.begin(), dataProcessorInfos.end(),
-                                         [&name = dp.name, &exec = currentWorkflow.executable](DataProcessorInfo const& info) { return (info.name == name && info.executable != exec); });
-      if (found == physicalWorkflow.end() || duplicate_name != dataProcessorInfos.end()) {
+      auto found = std::find_if(dataProcessorInfos.begin(), dataProcessorInfos.end(),
+                                [&name = dp.name](DataProcessorInfo const& info) { return info.name == name; });
+      // also adding duplicate names from different executables to the workflow to trigger the exception in the MATERIALISE_WORKFLOW state in case of duplicate names
+      if (found == dataProcessorInfos.end() || (found != dataProcessorInfos.end() && (*found).executable != currentWorkflow.executable)) {
         physicalWorkflow.push_back(dp);
         rankIndex.insert(std::make_pair(dp.name, workflowHashB));
       }