common/async: Completion removes unnecessary bind_executor()

destroy_defer() has already called bind_and_forward() to bind the
handler's associated executor

Signed-off-by: Casey Bodley <cbodley@redhat.com>

diff --git a/src/common/async/completion.h b/src/common/async/completion.h
index 3c310d7f833082157aa1ad1f3524d146130e07f3..6cdfaaa63b730a7de32da4228037cfb7233e5d85 100644 (file)
--- a/src/common/async/completion.h
+++ b/src/common/async/completion.h
@@ -202,7 +202,7 @@ class CompletionImpl final : public Completion<void(Args...), T> {
     auto f = bind_and_forward(ex2, std::move(handler), std::move(args));
     RebindTraits2::destroy(alloc2, this);
     RebindTraits2::deallocate(alloc2, this, 1);
-    boost::asio::defer(boost::asio::bind_executor(ex2, std::move(f)));
+    boost::asio::defer(std::move(f));
   }
   void destroy_dispatch(std::tuple<Args...>&& args) override {
     auto w = std::move(work);