Merge pull request #84 from alibuild/alibot-cleanup-13947

AliceO2Group · Feb 7, 2025 · d3762a4 · d3762a4
2 parents 5eed79c + f271c3a
commit d3762a4
Show file tree

Hide file tree

Showing 6 changed files with 19 additions and 13 deletions.
diff --git a/Framework/Core/include/Framework/AnalysisHelpers.h b/Framework/Core/include/Framework/AnalysisHelpers.h
@@ -191,7 +191,7 @@ struct Produces : WritingCursor<T> {
 };
 
 template <typename T>
-concept is_produces = requires (T t) { typename T::cursor_t; typename T::persistent_table_t; &T::cursor; };
+concept is_produces = requires(T t) { typename T::cursor_t; typename T::persistent_table_t; &T::cursor; };
 
 /// Use this to group together produces. Useful to separate them logically
 /// or simply to stay within the 100 elements per Task limit.
@@ -572,7 +572,7 @@ struct OutputObj {
 };
 
 template <typename T>
-concept is_outputobj = requires (T t) { &T::setObject; std::same_as<decltype(t.object), std::shared_ptr<typename T::obj_t>>; };
+concept is_outputobj = requires(T t) { &T::setObject; std::same_as<decltype(t.object), std::shared_ptr<typename T::obj_t>>; };
 
 /// This helper allows you to fetch a Sevice from the context or
 /// by using some singleton. This hopefully will hide the Singleton and
@@ -593,7 +593,7 @@ struct Service {
 };
 
 template <typename T>
-concept is_service = requires (T t) { std::same_as<decltype(t.service), typename T::service_t*>; &T::operator->;};
+concept is_service = requires(T t) { std::same_as<decltype(t.service), typename T::service_t*>; &T::operator->; };
 
 auto getTableFromFilter(soa::is_filtered_table auto const& table, soa::SelectionVector&& selection)
 {
@@ -721,7 +721,7 @@ struct Partition {
 };
 
 template <typename T>
-concept is_partition = requires (T t) {&T::updatePlaceholders; std::same_as<decltype(t.filter), expressions::Filter>; std::same_as<decltype(t.mFiltered), std::unique_ptr<o2::soa::Filtered<typename T::content_t>>>;};
+concept is_partition = requires(T t) {&T::updatePlaceholders; std::same_as<decltype(t.filter), expressions::Filter>; std::same_as<decltype(t.mFiltered), std::unique_ptr<o2::soa::Filtered<typename T::content_t>>>; };
 }  // namespace o2::framework
 
 namespace o2::soa

diff --git a/Framework/Core/include/Framework/AnalysisManagers.h b/Framework/Core/include/Framework/AnalysisManagers.h
@@ -31,7 +31,8 @@
 namespace o2::framework
 {
 
-namespace {
+namespace
+{
 template <typename O>
 static inline auto extractOriginal(ProcessingContext& pc)
 {
@@ -51,9 +52,10 @@ static inline auto extractOriginals(ProcessingContext& pc)
     return {pc.inputs().get<TableConsumer>(o2::aod::label<refs[Is]>())->asArrowTable()...};
   }(std::make_index_sequence<refs.size()>());
 }
-}
+} // namespace
 
-namespace analysis_task_parsers {
+namespace analysis_task_parsers
+{
 
 /// Options handling
 template <typename O>
@@ -442,7 +444,7 @@ void setPartition(P&, T&...)
 template <is_partition P, typename... T>
 void setPartition(P& partition, T&... tables)
 {
-  ([&](){ if constexpr (std::same_as<typename P::content_t, T>) {partition.bindTable(tables);} }(), ...);
+  ([&]() { if constexpr (std::same_as<typename P::content_t, T>) {partition.bindTable(tables);} }(), ...);
 }
 
 template <typename P, typename T>
@@ -507,7 +509,7 @@ static void setGroupedCombination(C& comb, TG& grouping, std::tuple<Ts...>& asso
   }
 }
 
-} // analysis_task_parsers
+} // namespace analysis_task_parsers
 
 template <typename ANY>
 struct UpdateProcessSwitches {

diff --git a/Framework/Core/include/Framework/AnalysisTask.h b/Framework/Core/include/Framework/AnalysisTask.h
@@ -580,7 +580,7 @@ DataProcessorSpec adaptAnalysisTask(ConfigContext const& ctx, Args&&... args)
       homogeneous_apply_refs([&eosContext](auto& element) {
           analysis_task_parsers::postRunService(eosContext, element);
           analysis_task_parsers::postRunOutput(eosContext, element);
-          return true;},
+          return true; },
                              *task.get());
       eosContext.services().get<ControlService>().readyToQuit(QuitRequest::Me);
     };

diff --git a/Framework/Core/include/Framework/Condition.h b/Framework/Core/include/Framework/Condition.h
@@ -43,7 +43,7 @@ struct Condition {
 };
 
 template <typename T>
-concept is_condition = requires (T t) {
+concept is_condition = requires(T t) {
   typename T::type;
   std::same_as<T*, decltype(t.instance)>;
   std::same_as<std::string, decltype(t.path)>;

diff --git a/Framework/Core/include/Framework/Configurable.h b/Framework/Core/include/Framework/Configurable.h
@@ -89,7 +89,11 @@ concept is_configurable = requires(T& t) { typename T::type; std::same_as<std::s
 using ConfigurableAxis = Configurable<std::vector<double>, ConfigParamKind::kAxisSpec, ConfigurablePolicyConst<std::vector<double>, ConfigParamKind::kAxisSpec>>;
 
 template <typename T>
-concept is_configurable_axis = is_configurable<T> && requires() {T::kind == ConfigParamKind::kAxisSpec;};
+concept is_configurable_axis = is_configurable<T>&&
+  requires()
+{
+  T::kind == ConfigParamKind::kAxisSpec;
+};
 
 template <typename R, typename T, typename... As>
 struct ProcessConfigurable : Configurable<bool, ConfigParamKind::kProcessFlag> {

diff --git a/Framework/Core/include/Framework/GroupedCombinations.h b/Framework/Core/include/Framework/GroupedCombinations.h
@@ -241,7 +241,7 @@ struct GroupedCombinationsGenerator {
 };
 
 template <typename T>
-concept is_combinations_generator = requires (T t) {
+concept is_combinations_generator = requires(T t) {
   typename T::GroupedIterator;
   &T::begin;
   &T::end;