connection: async_exec forwards completion token

cbodley · cbodley · commit 9ebcc544ae27 · 2023-08-02T11:45:57.000-04:00
async operations should support move-only completion handlers. forward the CompletionToken argument to avoid an unnecessary copy Fixes: #131 Signed-off-by: Casey Bodley <cbodley@redhat.com>
diff --git a/include/boost/redis/connection.hpp b/include/boost/redis/connection.hpp
@@ -227,9 +227,9 @@ class basic_connection {
    async_exec(
       request const& req,
       Response& resp = ignore,
-      CompletionToken token = CompletionToken{})
+      CompletionToken&& token = CompletionToken{})
    {
-      return impl_.async_exec(req, resp, token);
+      return impl_.async_exec(req, resp, std::forward<CompletionToken>(token));
    }
 
    /** @brief Cancel operations.

Original file line number	Diff line number	Diff line change
`@@ -227,9 +227,9 @@ class basic_connection {`
`227`	`227`	`async_exec(`
`228`	`228`	`request const& req,`
`229`	`229`	`Response& resp = ignore,`
`230`		`- CompletionToken token = CompletionToken{})`
	`230`	`+ CompletionToken&& token = CompletionToken{})`
`231`	`231`	`{`
`232`		`- return impl_.async_exec(req, resp, token);`
	`232`	`+ return impl_.async_exec(req, resp, std::forward<CompletionToken>(token));`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`/** @brief Cancel operations.`