fix: fix constructor

rime · Aug 30, 2018 · b25f968 · b25f968
1 parent 4a4a0cb
commit b25f968
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/WeaselIPC/PipeChannel.cpp b/WeaselIPC/PipeChannel.cpp
@@ -10,7 +10,7 @@ using namespace boost;
 #define _ThrowCode(__c) throw __c
 #define _ThrowIfNot(__c) { DWORD err; if ((err = ::GetLastError()) != __c) throw err; }
 
-PipeChannelBase::PipeChannelBase(std::wstring &pn_cmd, size_t bs = 4 * 1024, SECURITY_ATTRIBUTES *s = NULL)
+PipeChannelBase::PipeChannelBase(std::wstring &&pn_cmd, size_t bs = 4 * 1024, SECURITY_ATTRIBUTES *s = NULL)
 	: pname(pn_cmd),
 	write_stream(nullptr),
 	buff_size(bs),

diff --git a/WeaselIPCServer/WeaselServerImpl.cpp b/WeaselIPCServer/WeaselServerImpl.cpp
@@ -12,7 +12,7 @@ namespace weasel {
 		using Respond = std::function<void(Msg)>;
 		using ServerHandler = std::function<void(PipeMessage, Respond)>;
 
-		PipeServer(std::wstring &pn_cmd, SECURITY_ATTRIBUTES *s);
+		PipeServer(std::wstring &&pn_cmd, SECURITY_ATTRIBUTES *s);
 
 	public:
 		void Listen(ServerHandler const &handler);
@@ -335,8 +335,8 @@ void ServerImpl::HandlePipeMessage(PipeMessage pipe_msg, _Resp resp)
 	resp(result);
 }
 
-PipeServer::PipeServer(std::wstring &pn_cmd, SECURITY_ATTRIBUTES *s)
-	: PipeChannel(pn_cmd, s)
+PipeServer::PipeServer(std::wstring &&pn_cmd, SECURITY_ATTRIBUTES *s)
+	: PipeChannel(std::move(pn_cmd), s)
 {}
 
 void PipeServer::Listen(ServerHandler const &handler)

diff --git a/include/PipeChannel.h b/include/PipeChannel.h
@@ -11,7 +11,7 @@ namespace weasel {
 	public:
 		using Stream = boost::interprocess::wbufferstream;
 
-		PipeChannelBase(std::wstring &pn_cmd, size_t bs, SECURITY_ATTRIBUTES *s);
+		PipeChannelBase(std::wstring &&pn_cmd, size_t bs, SECURITY_ATTRIBUTES *s);
 		PipeChannelBase(PipeChannelBase &&r);
 		~PipeChannelBase();
 
@@ -71,7 +71,7 @@ namespace weasel {
 
 	public:
 		PipeChannel(std::wstring &&pn_cmd, SECURITY_ATTRIBUTES *s = NULL, size_t bs = 4 * 1024)
-			: PipeChannelBase(pn_cmd, bs, s)
+			: PipeChannelBase(std::move(pn_cmd), bs, s)
 		{}
 
 	public: