Merge pull request #130 from alibuild/alibot-cleanup-15213

Author: aalkin

Framework/Core/include/Framework/AnalysisHelpers.h

@@ -386,7 +386,7 @@ constexpr auto tableRef2OutputSpec()
   using md = typename o2::aod::MetadataTrait<o2::aod::Hash<R.desc_hash>>::metadata;
   if constexpr (soa::with_ccdb_urls<md>) {
     metadata.emplace_back("ccdb:", framework::VariantType::Bool, true, framework::ConfigParamSpec::HelpString{"\"\""});
-  } else if constexpr(soa::with_expression_pack<md>) {
+  } else if constexpr (soa::with_expression_pack<md>) {
     metadata.emplace_back("projectors", framework::VariantType::Bool, true, framework::ConfigParamSpec::HelpString{"\"\""});
   } else if constexpr (soa::with_index_pack<md>) {
     metadata.emplace_back("index-records", framework::VariantType::Bool, true, framework::ConfigParamSpec::HelpString{"\"\""});

Framework/Core/include/Framework/DataSpecViews.h

@@ -18,14 +18,14 @@ namespace o2::framework::checks
 {
 static auto has_params_with_name(std::string&& name)
 {
-  return [name](ConfigParamSpec const& p){return p.name.compare(name) == 0;};
+  return [name](ConfigParamSpec const& p) { return p.name.compare(name) == 0; };
 }
 
 static auto has_params_with_name_starting(std::string&& name)
 {
-  return [name](ConfigParamSpec const& p){return p.name.starts_with(name);};
-}
+  return [name](ConfigParamSpec const& p) { return p.name.starts_with(name); };
 }
+} // namespace o2::framework::checks
 
 namespace o2::framework::views
 {

Framework/Core/src/AnalysisSupportHelpers.cxx

@@ -175,13 +175,13 @@ void AnalysisSupportHelpers::addMissingOutputsToBuilder(std::vector<InputSpec> c
   // FIXME: until we have a single list of pairs
   additionalInputs |
     views::partial_match_filter(AODOrigins) |
-    std::ranges::views::filter([](InputSpec const& input){
-      return std::ranges::none_of(input.metadata, [](ConfigParamSpec const& p){ return (p.name.compare("projectors") == 0) || (p.name.compare("index-records") == 0); });
+    std::ranges::views::filter([](InputSpec const& input) {
+      return std::ranges::none_of(input.metadata, [](ConfigParamSpec const& p) { return (p.name.compare("projectors") == 0) || (p.name.compare("index-records") == 0); });
     }) |
     sinks::update_input_list{requestedAODs}; // update requestedAODs
   additionalInputs |
-    std::ranges::views::filter([](InputSpec const& input){
-      return std::ranges::any_of(input.metadata, [](ConfigParamSpec const& p){ return p.name.compare("projectors") == 0; });
+    std::ranges::views::filter([](InputSpec const& input) {
+      return std::ranges::any_of(input.metadata, [](ConfigParamSpec const& p) { return p.name.compare("projectors") == 0; });
     }) |
     sinks::update_input_list{requestedDYNs}; // update requestedDYNs
 }

Framework/Core/src/WorkflowHelpers.cxx

@@ -377,7 +377,7 @@ void WorkflowHelpers::injectServiceDevices(WorkflowSpec& workflow, ConfigContext
         dec.providedDYNs.emplace_back(output);
       } else if (hasCCDBURLs) {
         dec.providedTIMs.emplace_back(output);
-      } else if(hasIndexRecords) {
+      } else if (hasIndexRecords) {
         dec.providedIDXs.emplace_back(output);
       } else if (DataSpecUtils::partialMatch(output, AODOrigins)) {
         dec.providedAODs.emplace_back(output);