From d9f466b3b92f242343dcdecae83b40579f92c506 Mon Sep 17 00:00:00 2001 From: Ed Tanous Date: Mon, 6 Mar 2023 15:04:25 -0800 Subject: Take url views by value Any of our things taking URLs should be taking url_view by value, similar to how we take string_view. From the beast documentation: "...it acts like a string_view in terms of ownership." [1] Therefore, we should treat it like we treat string_view, and take by value, not reference. [1] https://www.boost.org/doc/libs/master/libs/url/doc/html/url/ref/boost__urls__url_view.html Tested: Stacked these patches. Redfish service validator passes. Signed-off-by: Ed Tanous Change-Id: I696b495f4aa04984225853f653cc175c0eaad79d --- .clang-tidy | 1 + 1 file changed, 1 insertion(+) (limited to '.clang-tidy') diff --git a/.clang-tidy b/.clang-tidy index bad6111fcb..4e09243ee4 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -316,3 +316,4 @@ CheckOptions: - { key: readability-identifier-naming.NamespaceCase, value: lower_case } - { key: readability-identifier-naming.StructCase, value: CamelCase } - { key: cppcoreguidelines-macro-usage.AllowedRegexp, value: DEBUG*|NLOHMANN_JSON_SERIALIZE_ENUM } + - { key: performance-unnecessary-value-param.AllowedTypes, value: ((segments_view)|(url_view)) } -- cgit v1.2.3