commit 5be2b14a2553f2aa0b64d1f88494b0561739d964
author Ed Tanous <ed@tanous.net> Wed Mar 27 15:27:04 2024 -0700
committer Ed Tanous <ed@tanous.net> Mon Apr 01 17:29:43 2024 +0000
tree 23226c6979b8ee0757b62ef890c54ce449533023
parent ddf3564e48bcee32dff1eff7a985a084be529615

Move where appropriate

Clang-tidy-18 has new checks that can find more cases where we've
missed an opportunity to std::move.

Fix them.

Tested: Logging works, unit tests pass.

Change-Id: I0cf58204ce7265828693b787a7b3a16484c3d5e5
Signed-off-by: Ed Tanous <ed@tanous.net>

http/utility.hpp

diff --git a/http/utility.hpp b/http/utility.hpp
index 47ddaf9..0476d73 100644
--- a/http/utility.hpp
+++ b/http/utility.hpp
@@ -439,7 +439,7 @@
 };
 
 inline bool readUrlSegments(boost::urls::url_view url,
-                            std::initializer_list<UrlSegment>&& segments)
+                            std::initializer_list<UrlSegment> segments)
 {
     boost::urls::segments_view urlSegments = url.segments();