diff options
| author | Sam Atkins <atkinssj@serenityos.org> | 2021-12-02 11:17:35 +0000 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2021-12-05 15:31:03 +0100 |
| commit | 92f8514a854233b9b533096006641d57a91d099a (patch) | |
| tree | 10429026787fc4e2c0bfb679c8f82ff8ba1eabd3 /Userland/Services/RequestServer/main.cpp | |
| parent | c67c1b583adec5fd9a40d5d69148c77e7e7acc3a (diff) | |
| download | serenity-92f8514a854233b9b533096006641d57a91d099a.zip | |
Services: Cast unused IPC::new_client_connection() results to void
These ones all manage their storage internally, whereas the WebContent
and ImageDecoder ones require the caller to manage their lifetime. This
distinction is not obvious to the user without looking through the code,
so an API that makes this clearer would be nice.
Diffstat (limited to 'Userland/Services/RequestServer/main.cpp')
| -rw-r--r-- | Userland/Services/RequestServer/main.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Userland/Services/RequestServer/main.cpp b/Userland/Services/RequestServer/main.cpp index 02c0474ddd..a97f26bc9b 100644 --- a/Userland/Services/RequestServer/main.cpp +++ b/Userland/Services/RequestServer/main.cpp @@ -37,7 +37,7 @@ ErrorOr<int> serenity_main(Main::Arguments) [[maybe_unused]] auto https = make<RequestServer::HttpsProtocol>(); auto socket = TRY(Core::LocalSocket::take_over_accepted_socket_from_system_server()); - IPC::new_client_connection<RequestServer::ClientConnection>(move(socket), 1); + (void)IPC::new_client_connection<RequestServer::ClientConnection>(move(socket), 1); auto result = event_loop.exec(); // FIXME: We exit instead of returning, so that protocol destructors don't get called. |