From: Sangwan Kwon <sangwan.kwon@samsung.com>
Date: Fri, 13 Dec 2019 05:04:11 +0000 (+0900)
Subject: Fix Protocol::Task signature
X-Git-Tag: submit/tizen/20200810.073515~132
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5f59dba324a6c81d969dba9234e7f84ff986a0c3;p=platform%2Fcore%2Fsecurity%2Fvist.git

Fix Protocol::Task signature

Signed-off-by: Sangwan Kwon <sangwan.kwon@samsung.com>
---

diff --git a/src/vist/rmi/exposer.cpp b/src/vist/rmi/exposer.cpp
index e33c515..603a476 100644
--- a/src/vist/rmi/exposer.cpp
+++ b/src/vist/rmi/exposer.cpp
@@ -36,7 +36,7 @@ class Exposer::Impl {
 public:
 	explicit Impl(Exposer& exposer, const std::string& path)
 	{
-		auto dispatcher = [&exposer](Message message) -> Message {
+		auto dispatcher = [&exposer](auto& message) -> Message {
 			std::string function = message.signature;
 			auto iter = exposer.functorMap.find(function);
 			if (iter == exposer.functorMap.end())
diff --git a/src/vist/rmi/remote.cpp b/src/vist/rmi/remote.cpp
index 7d24371..6ea77d7 100644
--- a/src/vist/rmi/remote.cpp
+++ b/src/vist/rmi/remote.cpp
@@ -38,7 +38,7 @@ public:
 	{
 	}
 
-	Message request(Message message)
+	Message request(Message& message)
 	{
 		return this->client.request(message);
 	}
@@ -53,7 +53,7 @@ Remote::Remote(const std::string& path) : pImpl(new Impl(path))
 
 Remote::~Remote() = default;
 
-Message Remote::request(Message message)
+Message Remote::request(Message& message)
 {
 	return pImpl->request(message);
 }
diff --git a/src/vist/rmi/remote.hpp b/src/vist/rmi/remote.hpp
index ae05319..63d1316 100644
--- a/src/vist/rmi/remote.hpp
+++ b/src/vist/rmi/remote.hpp
@@ -47,7 +47,7 @@ public:
 	R invoke(const std::string& name, Args&&... args);
 
 private:
-	Message request(Message message);
+	Message request(Message& message);
 
 	class Impl;
 	std::unique_ptr<Impl> pImpl;
diff --git a/src/vist/transport/protocol.hpp b/src/vist/transport/protocol.hpp
index 15b9352..b099acd 100644
--- a/src/vist/transport/protocol.hpp
+++ b/src/vist/transport/protocol.hpp
@@ -35,7 +35,7 @@ struct Protocol {
 	using Context = boost::asio::io_service;
 	using Endpoint = boost::asio::local::stream_protocol::endpoint;
 	using Socket = boost::asio::local::stream_protocol::socket;
-	using Task = std::function<Message(Message)>;
+	using Task = std::function<Message(Message&)>;
 
 	static Message Recv(Socket& socket);
 	static void Send(Socket& socket, Message& message);
diff --git a/src/vist/transport/tests/protocol.cpp b/src/vist/transport/tests/protocol.cpp
index 73c5140..5b1d685 100644
--- a/src/vist/transport/tests/protocol.cpp
+++ b/src/vist/transport/tests/protocol.cpp
@@ -115,7 +115,7 @@ TEST(ProtocolTests, async_server_sync_client)
 	auto async = std::make_shared<Protocol::Async>(context);
 	auto handler = [&](const auto& error) {
 		EXPECT_EQ(error, boost::system::errc::success);
-		auto task = [&](Message message) -> Message {
+		auto task = [&](auto& message) -> Message {
 			EXPECT_EQ(message.signature, requestSignature);
 
 			int recv1;
diff --git a/src/vist/transport/tests/server-client.cpp b/src/vist/transport/tests/server-client.cpp
index da316f1..d76c6f9 100644
--- a/src/vist/transport/tests/server-client.cpp
+++ b/src/vist/transport/tests/server-client.cpp
@@ -47,7 +47,7 @@ TEST(ServerClientTests, server)
 {
 	std::string sockPath = "vist-test.sock";
 
-	auto task = [&](Message message) -> Message {
+	auto task = [&](Message& message) -> Message {
 		EXPECT_EQ(message.signature, requestSignature);
 
 		int recv1;