From bdac12761cd92960c3df83c932aa610f2322215f Mon Sep 17 00:00:00 2001 From: rubenwardy Date: Sun, 2 Jul 2017 20:25:22 +0100 Subject: CSM: Revert "[CSM] Add send_chat_message and run_server_chatcommand" Original PR: #5747. This reverts commit 39f4a2f607d44738d60db84eba4b30e3d7450204. --- src/client.cpp | 52 +++-------------------------------------- src/client.h | 8 ------- src/defaultsettings.cpp | 1 - src/script/lua_api/l_client.cpp | 19 --------------- src/script/lua_api/l_client.h | 6 ----- 5 files changed, 3 insertions(+), 83 deletions(-) (limited to 'src') diff --git a/src/client.cpp b/src/client.cpp index 0f689a714..d0c90b108 100644 --- a/src/client.cpp +++ b/src/client.cpp @@ -83,7 +83,6 @@ Client::Client( m_con(PROTOCOL_ID, 512, CONNECTION_TIMEOUT, ipv6, this), m_address_name(address_name), m_server_ser_ver(SER_FMT_VER_INVALID), - m_last_chat_message_sent(time(NULL)), m_password(password), m_chosen_auth_mech(AUTH_MECHANISM_NONE), m_media_downloader(new ClientMediaDownloader()), @@ -388,14 +387,6 @@ void Client::step(float dtime) } } - /* - Send pending messages on out chat queue - */ - if (!m_out_chat_queue.empty() && canSendChatMessage()) { - sendChatMessage(m_out_chat_queue.front()); - m_out_chat_queue.pop(); - } - /* Handle environment */ @@ -1152,50 +1143,13 @@ void Client::sendInventoryAction(InventoryAction *a) Send(&pkt); } -bool Client::canSendChatMessage() const -{ - u32 now = time(NULL); - float time_passed = now - m_last_chat_message_sent; - - float virt_chat_message_allowance = m_chat_message_allowance + time_passed * - (CLIENT_CHAT_MESSAGE_LIMIT_PER_10S / 8.0f); - - if (virt_chat_message_allowance < 1.0f) - return false; - - return true; -} - void Client::sendChatMessage(const std::wstring &message) { - const s16 max_queue_size = g_settings->getS16("max_out_chat_queue_size"); - if (canSendChatMessage()) { - u32 now = time(NULL); - float time_passed = now - m_last_chat_message_sent; - m_last_chat_message_sent = time(NULL); - - m_chat_message_allowance += time_passed * (CLIENT_CHAT_MESSAGE_LIMIT_PER_10S / 8.0f); - if (m_chat_message_allowance > CLIENT_CHAT_MESSAGE_LIMIT_PER_10S) - m_chat_message_allowance = CLIENT_CHAT_MESSAGE_LIMIT_PER_10S; - - m_chat_message_allowance -= 1.0f; + NetworkPacket pkt(TOSERVER_CHAT_MESSAGE, 2 + message.size() * sizeof(u16)); - NetworkPacket pkt(TOSERVER_CHAT_MESSAGE, 2 + message.size() * sizeof(u16)); + pkt << message; - pkt << message; - - Send(&pkt); - } else if (m_out_chat_queue.size() < (u16) max_queue_size || max_queue_size == -1) { - m_out_chat_queue.push(message); - } else { - infostream << "Could not queue chat message because maximum out chat queue size (" - << max_queue_size << ") is reached." << std::endl; - } -} - -void Client::clearOutChatQueue() -{ - m_out_chat_queue = std::queue(); + Send(&pkt); } void Client::sendChangePassword(const std::string &oldpassword, diff --git a/src/client.h b/src/client.h index 0255b2803..29cbe9c10 100644 --- a/src/client.h +++ b/src/client.h @@ -40,8 +40,6 @@ with this program; if not, write to the Free Software Foundation, Inc., #include #include "filesys.h" -#define CLIENT_CHAT_MESSAGE_LIMIT_PER_10S 10.0f - struct MeshMakeData; class MapBlockMesh; class IWritableTextureSource; @@ -375,7 +373,6 @@ public: const StringMap &fields); void sendInventoryAction(InventoryAction *a); void sendChatMessage(const std::wstring &message); - void clearOutChatQueue(); void sendChangePassword(const std::string &oldpassword, const std::string &newpassword); void sendDamage(u8 damage); @@ -582,8 +579,6 @@ private: inline std::string getPlayerName() { return m_env.getLocalPlayer()->getName(); } - bool canSendChatMessage() const; - float m_packetcounter_timer = 0.0f; float m_connection_reinit_timer = 0.1f; float m_avg_rtt_timer = 0.0f; @@ -630,9 +625,6 @@ private: //s32 m_daynight_i; //u32 m_daynight_ratio; std::queue m_chat_queue; - std::queue m_out_chat_queue; - u32 m_last_chat_message_sent; - float m_chat_message_allowance = 5.0f; // The authentication methods we can use to enter sudo mode (=change password) u32 m_sudo_auth_methods; diff --git a/src/defaultsettings.cpp b/src/defaultsettings.cpp index 707f2c3aa..45920330f 100644 --- a/src/defaultsettings.cpp +++ b/src/defaultsettings.cpp @@ -57,7 +57,6 @@ void set_default_settings(Settings *settings) settings->setDefault("curl_verify_cert", "true"); settings->setDefault("enable_remote_media_server", "true"); settings->setDefault("enable_client_modding", "false"); - settings->setDefault("max_out_chat_queue_size", "20"); // Keymap settings->setDefault("remote_port", "30000"); diff --git a/src/script/lua_api/l_client.cpp b/src/script/lua_api/l_client.cpp index 1077d5f2d..0b957a6e2 100644 --- a/src/script/lua_api/l_client.cpp +++ b/src/script/lua_api/l_client.cpp @@ -85,23 +85,6 @@ int ModApiClient::l_display_chat_message(lua_State *L) return 1; } -// send_chat_message(message) -int ModApiClient::l_send_chat_message(lua_State *L) -{ - if (!lua_isstring(L, 1)) - return 0; - std::string message = luaL_checkstring(L, 1); - getClient(L)->sendChatMessage(utf8_to_wide(message)); - return 0; -} - -// clear_out_chat_queue() -int ModApiClient::l_clear_out_chat_queue(lua_State *L) -{ - getClient(L)->clearOutChatQueue(); - return 0; -} - // get_player_names() int ModApiClient::l_get_player_names(lua_State *L) { @@ -354,8 +337,6 @@ void ModApiClient::Initialize(lua_State *L, int top) API_FCT(get_current_modname); API_FCT(print); API_FCT(display_chat_message); - API_FCT(send_chat_message); - API_FCT(clear_out_chat_queue); API_FCT(get_player_names); API_FCT(set_last_run_mod); API_FCT(get_last_run_mod); diff --git a/src/script/lua_api/l_client.h b/src/script/lua_api/l_client.h index 7472915f5..6bb12187f 100644 --- a/src/script/lua_api/l_client.h +++ b/src/script/lua_api/l_client.h @@ -37,12 +37,6 @@ private: // display_chat_message(message) static int l_display_chat_message(lua_State *L); - // send_chat_message(message) - static int l_send_chat_message(lua_State *L); - - // clear_out_chat_queue() - static int l_clear_out_chat_queue(lua_State *L); - // get_player_names() static int l_get_player_names(lua_State *L); -- cgit v1.2.3