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 e33c515b8e06b0832f4df4fbf72c6bcdb9e10cb6..603a476ffd70ddebc31f85ff0564756d76872a2b 100644 (file)
--- 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 7d243713fd7ce83f384755dd425c20b4e04b46de..6ea77d7ddc17f6a754a441c8f60364ca701ee619 100644 (file)
--- 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 ae05319057bc955f0ac20d80ea78ea6803d7a82a..63d131616174937c27420773f3e450ce2097cb7e 100644 (file)
--- 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 15b9352a48582426f296cedd0796dfc99db05734..b099acd5de9fff576b1d7c9d08954a77794e64c3 100644 (file)
--- 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 73c51408f62595ef8da33d2912a9e7dd89d04c67..5b1d685d97794044ba03667571913aa33c9ecc9e 100644 (file)
--- 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 da316f1a1a6650b2c636fca13419308fa47e9de0..d76c6f9b205a5ceb878ca7d3f580567b96d8c273 100644 (file)
--- 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;