wsd/DocumentBroker.cpp | 19 ++++++------------- wsd/DocumentBroker.hpp | 2 +- 2 files changed, 7 insertions(+), 14 deletions(-)
New commits: commit 5d057de0dd7a4d1e5d5cf5fe995f021110691aca Author: Pranav Kant <[email protected]> Date: Wed Jun 21 15:37:38 2017 +0530 Bin unused parameter, rude Change-Id: I39f55f55f8a0c5432350625d9720ea2367955a8c Reviewed-on: https://gerrit.libreoffice.org/39085 Reviewed-by: pranavk <[email protected]> Tested-by: pranavk <[email protected]> (cherry picked from commit c21de445247813c27f66c62a9bf6af03a11386ec) Reviewed-on: https://gerrit.libreoffice.org/39703 Reviewed-by: Michael Meeks <[email protected]> Tested-by: Michael Meeks <[email protected]> diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp index bd506fc2..7d482965 100644 --- a/wsd/DocumentBroker.cpp +++ b/wsd/DocumentBroker.cpp @@ -283,7 +283,7 @@ void DocumentBroker::pollThread() } // Terminate properly while we can. - terminateChild(closeReason, false); + terminateChild(closeReason); // Stop to mark it done and cleanup. _poll->stop(); @@ -1369,17 +1369,14 @@ void DocumentBroker::childSocketTerminated() shutdownClients("terminated"); } -void DocumentBroker::terminateChild(const std::string& closeReason, const bool rude) +void DocumentBroker::terminateChild(const std::string& closeReason) { assertCorrectThread(); LOG_INF("Terminating doc [" << _docKey << "]."); // Close all running sessions - if (!rude) - { - shutdownClients(closeReason); - } + shutdownClients(closeReason); if (_childProcess) { @@ -1387,12 +1384,8 @@ void DocumentBroker::terminateChild(const std::string& closeReason, const bool r // First flag to stop as it might be waiting on our lock // to process some incoming message. - if (!rude) - { - _childProcess->stop(); - } - - _childProcess->close(rude); + _childProcess->stop(); + _childProcess->close(false); } _stop = true; @@ -1403,7 +1396,7 @@ void DocumentBroker::closeDocument(const std::string& reason) assertCorrectThread(); LOG_DBG("Closing DocumentBroker for docKey [" << _docKey << "] with reason: " << reason); - terminateChild(reason, true); + terminateChild(reason); } void DocumentBroker::updateLastActivityTime() diff --git a/wsd/DocumentBroker.hpp b/wsd/DocumentBroker.hpp index 27d525c0..11354c0f 100644 --- a/wsd/DocumentBroker.hpp +++ b/wsd/DocumentBroker.hpp @@ -344,7 +344,7 @@ private: /// This gracefully terminates the connection /// with the child and cleans up ChildProcess etc. - void terminateChild(const std::string& closeReason, const bool rude); + void terminateChild(const std::string& closeReason); /// Saves the doc to the storage. bool saveToStorageInternal(const std::string& sesionId, bool success, const std::string& result = ""); _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
