commit 593f6449a6afa130a28c4cc9a55fb857e019917e
author Nan Zhou <nanzhoumails@gmail.com> Mon Jun 06 01:31:59 2022 +0000
committer Ed Tanous <ed@tanous.net> Fri Jun 10 01:45:51 2022 +0000
tree 89200605e45b3a930337a9114ed9fa6827c5b6ea
parent 3215e700d225fd56bcf036d2095e7157bd38f03a

query params: avoid copy

|processAllParams| now can take reference of parsed |Query| given that
it's read-only now. The only copy is kept by the lambda.

Tested:
1. on my mock environment, query parameter works as expected. Tested
$only, $expand, $top, and $skip.

Signed-off-by: Nan Zhou <nanzhoumails@gmail.com>
Change-Id: I1edf53d3b0e030c7370eb7ba937644d5ced074dc

redfish-core/include/utils/query_param.hpp

diff --git a/redfish-core/include/utils/query_param.hpp b/redfish-core/include/utils/query_param.hpp
index 9f17b7b..6c636f3 100644
--- a/redfish-core/include/utils/query_param.hpp
+++ b/redfish-core/include/utils/query_param.hpp
@@ -569,7 +569,7 @@
 }
 
 inline void
-    processAllParams(crow::App& app, const Query query,
+    processAllParams(crow::App& app, const Query& query,
                      std::function<void(crow::Response&)>& completionHandler,
                      crow::Response& intermediateResponse)
 {