diff options
author | Ed Tanous <edtanous@google.com> | 2023-02-16 21:28:44 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2023-02-24 20:21:14 +0300 |
commit | 26ccae32112679c4653c1e3f8a1203c828bea05c (patch) | |
tree | a87d5056294e163b9ddeed9e867bcfb0750a8bb5 /http/routing.hpp | |
parent | 7da1c58890e82194ff83ca6c8d55b5c327f2444a (diff) | |
download | bmcweb-26ccae32112679c4653c1e3f8a1203c828bea05c.tar.xz |
Pass string views by value
string_view should always be passed by value; This commit is a sed
replace of the code to make all string_views pass by value, per general
coding guidelines[1].
[1] https://quuxplusone.github.io/blog/2021/11/09/pass-string-view-by-value/
Tested: Code compiles.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I55b342a29a0fbfce0a4ed9ea63db6014d03b134c
Diffstat (limited to 'http/routing.hpp')
-rw-r--r-- | http/routing.hpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/http/routing.hpp b/http/routing.hpp index 5b3ddfe429..ae63f0504c 100644 --- a/http/routing.hpp +++ b/http/routing.hpp @@ -422,7 +422,7 @@ struct RuleParameterTraits return *p; } - self_t& name(const std::string_view name) noexcept + self_t& name(std::string_view name) noexcept { self_t* self = static_cast<self_t*>(this); self->nameStr = name; @@ -663,7 +663,7 @@ class TaggedRule : } template <typename Func> - void operator()(const std::string_view name, Func&& f) + void operator()(std::string_view name, Func&& f) { nameStr = name; (*this).template operator()<Func>(std::forward(f)); @@ -807,7 +807,7 @@ class Trie } std::pair<unsigned, RoutingParams> - find(const std::string_view reqUrl, const Node* node = nullptr, + find(std::string_view reqUrl, const Node* node = nullptr, size_t pos = 0, RoutingParams* params = nullptr) const { RoutingParams empty; |