use early return

AliceO2Group · Nov 15, 2024 · 1546f60 · 1546f60
1 parent 84510eb
commit 1546f60
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/Framework/Core/include/Framework/ASoA.h b/Framework/Core/include/Framework/ASoA.h
@@ -2024,11 +2024,16 @@ ColumnGetterFunction<R, T> createGetterPtr(const std::string_view& columnLabel)
 {
   // allows user to use consistent formatting (with prefix) of all column labels
   // by default there isn't 'f' prefix for dynamic column labels
-  bool isPrefixMatch = columnLabel.size() > 1 && columnLabel.substr(1) == C::columnLabel();
+  if(columnLabel.size() > 1 && columnLabel.substr(1) == C::columnLabel()) {
+    return &getColumnValue<R, T, C>;
+  }
+
   // check also exact match if user is aware of prefix missing
-  bool isExactMatch = columnLabel == C::columnLabel();
+  if(columnLabel == C::columnLabel()) {
+    return &getColumnValue<R, T, C>;
+  }
 
-  return (isPrefixMatch || isExactMatch) ? &getColumnValue<R, T, C> : nullptr;
+  return nullptr;
 }
 
 template <typename R, typename T, typename... Cs>