[Libreoffice-commits] online.git: loolwsd/Admin.cpp loolwsd/DocumentBroker.cpp

Tor Lillqvist tml at collabora.com
Thu Sep 22 06:04:38 UTC 2016


 loolwsd/Admin.cpp          |    2 +-
 loolwsd/DocumentBroker.cpp |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 39f4e92b022893a609ad527e082d2124a67cfceb
Author: Tor Lillqvist <tml at collabora.com>
Date:   Thu Sep 22 08:59:23 2016 +0300

    Log::end was missing in two places that used Log::debug()
    
    It's too easy to forget the Log::end when using the Log::debug() etc
    API. Not sure what could be done to that. Change the logging API to be
    like LibreOffice's SAL_INFO?

diff --git a/loolwsd/Admin.cpp b/loolwsd/Admin.cpp
index 00dfc49..bf0423f 100644
--- a/loolwsd/Admin.cpp
+++ b/loolwsd/Admin.cpp
@@ -225,7 +225,7 @@ void AdminRequestHandler::handleWSRequests(HTTPServerRequest& request, HTTPServe
                             []() { },
                             []() { return TerminationFlag.load(); });
 
-    Log::debug() << "Finishing Admin Session " << Util::encodeId(sessionId);
+    Log::debug() << "Finishing Admin Session " << Util::encodeId(sessionId) << Log::end;
 }
 
 AdminRequestHandler::AdminRequestHandler(Admin* adminManager)
diff --git a/loolwsd/DocumentBroker.cpp b/loolwsd/DocumentBroker.cpp
index 9f630bf..989e056 100644
--- a/loolwsd/DocumentBroker.cpp
+++ b/loolwsd/DocumentBroker.cpp
@@ -229,7 +229,7 @@ bool DocumentBroker::save(bool success, const std::string& result)
     // notify the waiting thread, if any.
     if (!success && result == "unmodified")
     {
-        Log::debug() << "Save skipped as document was not modified";
+        Log::debug("Save skipped as document was not modified");
         _saveCV.notify_all();
         return true;
     }


More information about the Libreoffice-commits mailing list