diff options
author | Tiger Wang <ziwei.tiger@outlook.com> | 2020-07-23 01:34:43 +0200 |
---|---|---|
committer | Tiger Wang <ziwei.tiger@outlook.com> | 2020-09-05 14:16:52 +0200 |
commit | 1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b (patch) | |
tree | ea7f02576728ac4143707b0af3829ae843bde041 /src/Server.cpp | |
parent | Rename AddEntityIfNotPresent to AddPlayer (diff) | |
download | cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar.gz cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar.bz2 cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar.lz cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar.xz cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.tar.zst cuberite-1bc12ba2b3bc2c4fbe8fad4797cf33266eb13f4b.zip |
Diffstat (limited to 'src/Server.cpp')
-rw-r--r-- | src/Server.cpp | 48 |
1 files changed, 30 insertions, 18 deletions
diff --git a/src/Server.cpp b/src/Server.cpp index d9a0ced27..a1dedf333 100644 --- a/src/Server.cpp +++ b/src/Server.cpp @@ -94,7 +94,7 @@ void cServer::cTickThread::Execute(void) { auto NowTime = std::chrono::steady_clock::now(); auto msec = std::chrono::duration_cast<std::chrono::milliseconds>(NowTime - LastTime).count(); - m_ShouldTerminate = !m_Server.Tick(static_cast<float>(msec)); + m_Server.Tick(static_cast<float>(msec)); auto TickTime = std::chrono::steady_clock::now() - NowTime; if (TickTime < msPerTick) @@ -118,7 +118,6 @@ cServer::cServer(void) : m_PlayerCount(0), m_ClientViewDistance(0), m_bIsConnected(false), - m_bRestarting(false), m_RCONServer(*this), m_MaxPlayers(0), m_bIsHardcore(false), @@ -331,27 +330,16 @@ cTCPLink::cCallbacksPtr cServer::OnConnectionAccepted(const AString & a_RemoteIP -bool cServer::Tick(float a_Dt) +void cServer::Tick(float a_Dt) { // Send the tick to the plugins, as well as let the plugin manager reload, if asked to (issue #102): cPluginManager::Get()->Tick(a_Dt); - // Let the Root process all the queued commands: - cRoot::Get()->TickCommands(); + // Process all the queued commands: + TickCommands(); // Tick all clients not yet assigned to a world: TickClients(a_Dt); - - if (!m_bRestarting) - { - return true; - } - else - { - m_bRestarting = false; - m_RestartEvent.Set(); - return false; - } } @@ -445,6 +433,17 @@ bool cServer::Command(cClientHandle & a_Client, AString & a_Cmd) +void cServer::QueueExecuteConsoleCommand(const AString & a_Cmd, cCommandOutputCallback & a_Output) +{ + // Put the command into a queue (Alleviates FS #363): + cCSLock Lock(m_CSPendingCommands); + m_PendingCommands.emplace_back(a_Cmd, &a_Output); +} + + + + + void cServer::ExecuteConsoleCommand(const AString & a_Cmd, cCommandOutputCallback & a_Output) { AStringVector split = StringSplit(a_Cmd, " "); @@ -647,8 +646,7 @@ void cServer::Shutdown(void) m_ServerHandles.clear(); // Notify the tick thread and wait for it to terminate: - m_bRestarting = true; - m_RestartEvent.Wait(); + m_TickThread.Stop(); cRoot::Get()->SaveAllChunks(); @@ -706,3 +704,17 @@ void cServer::AuthenticateUser(int a_ClientID, const AString & a_Name, const cUU +void cServer::TickCommands(void) +{ + decltype(m_PendingCommands) PendingCommands; + { + cCSLock Lock(m_CSPendingCommands); + std::swap(PendingCommands, m_PendingCommands); + } + + // Execute any pending commands: + for (const auto & Command : PendingCommands) + { + ExecuteConsoleCommand(Command.first, *Command.second); + } +} |