From de482bdfd0233c0f5c2a0eac2cd2dd0b55ad9c72 Mon Sep 17 00:00:00 2001 From: Rim Date: Wed, 15 May 2024 15:16:26 -0400 Subject: [PATCH] fix(elimination_handler): re-write patch --- .gitmodules | 2 +- src/server.cpp | 14 ++++++-------- src/server_list.hpp | 4 ++-- src/services/elimination_handler.cpp | 14 ++++++-------- 4 files changed, 15 insertions(+), 19 deletions(-) diff --git a/.gitmodules b/.gitmodules index 588e7b6..80688d5 100644 --- a/.gitmodules +++ b/.gitmodules @@ -19,4 +19,4 @@ [submodule "deps/curl"] path = deps/curl url = https://github.com/curl/curl.git - branch = curl-8_5_0 + branch = curl-8_7_1 \ No newline at end of file diff --git a/src/server.cpp b/src/server.cpp index 77f293a..96a4f28 100644 --- a/src/server.cpp +++ b/src/server.cpp @@ -57,11 +57,11 @@ void server::run_frame() } catch (const service::execution_exception& e) { - console::warn("Execption in service: %s", e.what()); + console::warn("Exception in service: %s", e.what()); } catch (const std::exception& e) { - console::error("Fatal execption in service: %s", e.what()); + console::error("Fatal exception in service: %s", e.what()); } } } @@ -84,8 +84,7 @@ void server::handle_command(const network::address& target, const std::string_vi } #ifdef DEBUG - console::log("Handling command (%s): %.*s - %.*s", target.to_string().data(), command.size(), command.data(), - data.size(), data.data()); + console::log("Handling command (%s): %.*s - %.*s", target.to_string().data(), command.size(), command.data(), data.size(), data.data()); #endif try @@ -94,12 +93,11 @@ void server::handle_command(const network::address& target, const std::string_vi } catch (const service::execution_exception& e) { - console::warn("Execption in command %.*s (%s): %s", command.size(), command.data(), target.to_string().data(), + console::warn("Exception in command %.*s (%s): %s", command.size(), command.data(), target.to_string().data(), e.what()); } catch (const std::exception& e) { - console::error("Fatal execption in command %.*s (%s): %s", command.size(), command.data(), - target.to_string().data(), e.what()); + console::error("Fatal exception in command %.*s (%s): %s", command.size(), command.data(), target.to_string().data(), e.what()); } -} +} \ No newline at end of file diff --git a/src/server_list.hpp b/src/server_list.hpp index 37de262..0cd6302 100644 --- a/src/server_list.hpp +++ b/src/server_list.hpp @@ -3,11 +3,11 @@ #include "game_server.hpp" #include "network_list.hpp" -class server_list : public network_list +class server_list : public network_list { public: void find_registered_servers(game_type game, int protocol, const access_func& accessor); void find_registered_servers(game_type game, int protocol, const const_access_func& accessor) const; void heartbeat(const network::address& address); -}; +}; \ No newline at end of file diff --git a/src/services/elimination_handler.cpp b/src/services/elimination_handler.cpp index fc41ebd..f9d7236 100644 --- a/src/services/elimination_handler.cpp +++ b/src/services/elimination_handler.cpp @@ -7,7 +7,7 @@ constexpr size_t MAX_SERVERS_PER_GAME = 10; void elimination_handler::run_frame() { - std::unordered_map> server_count; + std::unordered_map> server_count; auto now = std::chrono::high_resolution_clock::now(); this->get_server().get_server_list().iterate([&](server_list::iteration_context& context) @@ -29,17 +29,15 @@ void elimination_handler::run_frame() if (server.game == game_type::t7 && server.protocol < T7_PROTOCOL) { #ifdef _DEBUG - console::log("Removing T7 server '%s' because they are using an outdated protocol (%i)\n", - context.get_address().to_string().data(), server.protocol); + console::log("Removing T7 server '%s' because they are using an outdated protocol (%i)", context.get_address().to_string().data(), server.protocol); #endif context.remove(); } - ++server_count[server.game][context.get_address()]; - if (server_count[server.game][context.get_address()] >= MAX_SERVERS_PER_GAME) + ++server_count[server.game][context.get_address().to_string(false)]; + if (server_count[server.game][context.get_address().to_string(false)] >= MAX_SERVERS_PER_GAME) { - console::log("Removing server '%s' because it exceeds MAX_SERVERS_PER_GAME\n", - context.get_address().to_string().data()); + console::log("Removing server '%s' because it exceeds MAX_SERVERS_PER_GAME", context.get_address().to_string().data()); context.remove(); } }); @@ -55,4 +53,4 @@ void elimination_handler::run_frame() context.remove(); } }); -} +} \ No newline at end of file