diff options
| author | Ed Tanous <edtanous@google.com> | 2023-03-07 02:04:25 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2023-03-15 18:52:56 +0300 |
| commit | d9f466b3b92f242343dcdecae83b40579f92c506 (patch) | |
| tree | e59b36731a8b3f26dcdab50b0e889d217a849114 /.clang-tidy | |
| parent | 6ce82fabeba1f2549b98c3ad2f8542c6879682c8 (diff) | |
| download | bmcweb-d9f466b3b92f242343dcdecae83b40579f92c506.tar.xz | |
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 <edtanous@google.com>
Change-Id: I696b495f4aa04984225853f653cc175c0eaad79d
Diffstat (limited to '.clang-tidy')
| -rw-r--r-- | .clang-tidy | 1 |
1 files changed, 1 insertions, 0 deletions
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)) } |