From 45553e11c965db218733f9ad32ecde391b393443 Mon Sep 17 00:00:00 2001
From: Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>
Date: Thu, 5 Jan 2023 17:54:08 +0000
Subject: [PATCH] refactor: Make `ThreadHTTP` return void

The `bool` return value was introduced in 755aa05174e06effd758eeb78c5af9fb465e9611.

It has been not used since 8d3f46ec3938e2ba17654fecacd1d2629f9915fd.

No behavior change.
---
 src/httpserver.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/httpserver.cpp b/src/httpserver.cpp
index 6597ed71af1..720f5c9353b 100644
--- a/src/httpserver.cpp
+++ b/src/httpserver.cpp
@@ -279,7 +279,7 @@ static void http_reject_request_cb(struct evhttp_request* req, void*)
 }
 
 /** Event dispatcher thread */
-static bool ThreadHTTP(struct event_base* base)
+static void ThreadHTTP(struct event_base* base)
 {
     util::ThreadRename("http");
     SetSyscallSandboxPolicy(SyscallSandboxPolicy::NET_HTTP_SERVER);
@@ -287,7 +287,6 @@ static bool ThreadHTTP(struct event_base* base)
     event_base_dispatch(base);
     // Event loop will be interrupted by InterruptHTTPServer()
     LogPrint(BCLog::HTTP, "Exited http event loop\n");
-    return event_base_got_break(base) == 0;
 }
 
 /** Bind HTTP server to specified addresses */