From 54e8b6628cf7e4014e4e5b311d936c52e898f5f1 Mon Sep 17 00:00:00 2001
From: fakeshadow <24548779@qq.com>
Date: Wed, 23 Dec 2020 03:18:58 +0800
Subject: [PATCH] use FnOnce instead of Fn

---
 actix-server/src/builder.rs       | 2 +-
 actix-server/tests/test_server.rs | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/actix-server/src/builder.rs b/actix-server/src/builder.rs
index 6449cd0f..3572f3a7 100644
--- a/actix-server/src/builder.rs
+++ b/actix-server/src/builder.rs
@@ -299,7 +299,7 @@ impl ServerBuilder {
 
     pub fn on_stop<F, Fut>(mut self, future: F) -> Self
     where
-        F: Fn() -> Fut + 'static,
+        F: FnOnce() -> Fut + 'static,
         Fut: Future<Output = ()>,
     {
         self.on_stop = Box::pin(async move { future().await });
diff --git a/actix-server/tests/test_server.rs b/actix-server/tests/test_server.rs
index 6391e64a..eb6c75a6 100644
--- a/actix-server/tests/test_server.rs
+++ b/actix-server/tests/test_server.rs
@@ -201,11 +201,8 @@ fn test_on_stop() {
             let srv: Server = Server::build()
                 .backlog(100)
                 .disable_signals()
-                .on_stop(move || {
-                    let bool = bool.clone();
-                    async move {
-                        bool.store(true, Ordering::SeqCst);
-                    }
+                .on_stop(|| async move {
+                    bool.store(true, Ordering::SeqCst);
                 })
                 .bind("test", addr, move || {
                     fn_service(|io: TcpStream| async move {