Minor improvements

XRPLF · Oct 13, 2023 · 355cf6c · 355cf6c
1 parent e3d382f
commit 355cf6c
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 17 deletions.
diff --git a/src/ripple/rpc/impl/Handler.cpp b/src/ripple/rpc/impl/Handler.cpp
@@ -22,6 +22,8 @@
 #include <ripple/rpc/impl/Handler.h>
 #include <ripple/rpc/impl/RPCHelpers.h>
 
+#include <map>
+
 namespace ripple {
 namespace RPC {
 namespace {
@@ -57,6 +59,19 @@ handle(JsonContext& context, Object& object)
     return status;
 };
 
+template <typename HandlerImpl>
+Handler
+handlerFrom()
+{
+    return {
+        HandlerImpl::name,
+        &handle<Json::Value, HandlerImpl>,
+        HandlerImpl::role,
+        HandlerImpl::condition,
+        HandlerImpl::minApiVer,
+        HandlerImpl::maxApiVer};
+}
+
 Handler const handlerArray[]{
     // Some handlers not specified here are added to the table via addHandler()
     // Request-response methods
@@ -244,15 +259,15 @@ class HandlerTable
     std::vector<char const*>
     getHandlerNames() const
     {
-        static constexpr auto less = [](const char* const lh,
-                                        const char* const rh) -> bool {
-            return std::strcmp(lh, rh);
-        };
-        std::set<const char* const, decltype(less)> names;
+        std::vector<char const*> ret;
         for (auto const& i : table_)
-            names.insert(i.second.name_);
+        {
+            // Note, table_ is always ordered, allowing such a simple check
+            if (ret.empty() || std::strcmp(ret.back(), i.second.name_) != 0)
+                ret.push_back(i.second.name_);
+        }
 
-        return std::vector<char const*>{names.begin(), names.end()};
+        return ret;
     }
 
 private:
@@ -270,15 +285,7 @@ class HandlerTable
             HandlerImpl::minApiVer,
             HandlerImpl::maxApiVer));
 
-        Handler h;
-        h.name_ = HandlerImpl::name;
-        h.valueMethod_ = &handle<Json::Value, HandlerImpl>;
-        h.role_ = HandlerImpl::role;
-        h.condition_ = HandlerImpl::condition;
-        h.minApiVer_ = HandlerImpl::minApiVer;
-        h.maxApiVer_ = HandlerImpl::maxApiVer;
-
-        table_.insert({HandlerImpl::name, h});
+        table_.insert({HandlerImpl::name, handlerFrom<HandlerImpl>()});
     }
 };
 

diff --git a/src/ripple/rpc/impl/Handler.h b/src/ripple/rpc/impl/Handler.h
@@ -55,7 +55,7 @@ struct Handler
     RPC::Condition condition_;
 
     unsigned minApiVer_ = 1;
-    unsigned maxApiVer_ = RPC::apiMaximumValidVersion;
+    unsigned maxApiVer_ = apiMaximumValidVersion;
 };
 
 Handler const*