loolwsd/DocumentBroker.cpp |    4 +++-
 loolwsd/Makefile.am        |    3 +++
 2 files changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 827d5ed9c825c3636d32071de2611029436e1f87
Author: Miklos Vajna <vmik...@collabora.co.uk>
Date:   Fri Sep 30 09:29:08 2016 +0200

    DocumentBroker: it's enough to build that temporary string once
    
    Change-Id: I253874fefdb2dfe9bb22a8b3bef641499a49dcc9

diff --git a/loolwsd/DocumentBroker.cpp b/loolwsd/DocumentBroker.cpp
index ba2dd04..aeafaa3 100644
--- a/loolwsd/DocumentBroker.cpp
+++ b/loolwsd/DocumentBroker.cpp
@@ -450,9 +450,11 @@ void DocumentBroker::alertAllUsersOfDocument(const 
std::string& cmd, const std::
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
+    std::stringstream ss;
+    ss << "error: cmd=" << cmd << " kind=" << kind;
     for (auto& it: _sessions)
     {
-        it.second->sendTextFrame("error: cmd=" + cmd + " kind=" + kind);
+        it.second->sendTextFrame(ss.str());
     }
 }
 
diff --git a/loolwsd/Makefile.am b/loolwsd/Makefile.am
index b9c935c..fa6c3a0 100644
--- a/loolwsd/Makefile.am
+++ b/loolwsd/Makefile.am
@@ -170,6 +170,9 @@ clean-cache cache-clean:
 # Intentionally don't use "*" below... Avoid risk of accidentally running rm 
-rf /*
        test -n "@LOOLWSD_CACHEDIR@" && rm -rf "@LOOLWSD_CACHEDIR@"/[0-9a-f]
 
+clang-tidy:
+       for i in *.cpp; do echo $$i; clang-tidy -header-filter=^$(PWD).* $$i || 
break; done
+
 # After building loolforkit, set its capabilities as required. Do it
 # already after a plain 'make' to allow for testing without
 # installing. When building for packaging, no need for this, as the
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to