refactor: simplify Invoker::IsOK() (#45819)

trop[bot] · ckerr · web-flow · commit 4ed1ac6ab07c · 2025-02-27T00:46:14.000-06:00
* refactor: simplify Invoker::IsOK()

Co-authored-by: Charles Kerr &lt;charles@charleskerr.com&gt;

* refactor: might as well make it [[nodiscard]] as well

Co-authored-by: Charles Kerr &lt;charles@charleskerr.com&gt;

---------

Co-authored-by: trop[bot] &lt;37223003+trop[bot]@users.noreply.github.com&gt;
Co-authored-by: Charles Kerr &lt;charles@charleskerr.com&gt;
diff --git a/shell/common/gin_helper/function_template.h b/shell/common/gin_helper/function_template.h
@@ -262,7 +262,9 @@ class Invoker<std::index_sequence<indices...>, ArgTypes...>
       : ArgumentHolder<indices, ArgTypes>(args, invoker_options)...,
         args_(args) {}
 
-  bool IsOK() { return And(ArgumentHolder<indices, ArgTypes>::ok...); }
+  [[nodiscard]] bool IsOK() const {
+    return (... && ArgumentHolder<indices, ArgTypes>::ok);
+  }
 
   template <typename ReturnType>
   void DispatchToCallback(
@@ -285,12 +287,6 @@ class Invoker<std::index_sequence<indices...>, ArgTypes...>
   }
 
  private:
-  static bool And() { return true; }
-  template <typename... T>
-  static bool And(bool arg1, T... args) {
-    return arg1 && And(args...);
-  }
-
   raw_ptr<gin::Arguments> args_;
 };
 
