[Libreoffice-commits] online.git: common/FileUtil.hpp

Tor Lillqvist (via logerrit) logerrit at kemper.freedesktop.org
Mon Apr 20 13:35:23 UTC 2020


 common/FileUtil.hpp |   14 --------------
 1 file changed, 14 deletions(-)

New commits:
commit 5fff61ef14253c171e55ccafb5e89b93d6bbcf35
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Mon Apr 20 16:17:08 2020 +0300
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Mon Apr 20 15:35:05 2020 +0200

    There doesn't seem to be any use of FileUtil::alertAllUsers()
    
    Which is good, Util::alertAllUsers() is enough of a complicated mess.
    
    Change-Id: Ibac302ea8a7506baa992d71e3891b5764b6ed279
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92569
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tor Lillqvist <tml at collabora.com>

diff --git a/common/FileUtil.hpp b/common/FileUtil.hpp
index 29426e16c..77b96bf15 100644
--- a/common/FileUtil.hpp
+++ b/common/FileUtil.hpp
@@ -39,20 +39,6 @@ namespace FileUtil
     // We work around some of the mess of using the same sources both on the server side and in unit
     // tests with conditional compilation based on BUILDING_TESTS.
 
-#ifndef BUILDING_TESTS
-    // Send a 'error:' message with the specified cmd and kind parameters to all connected
-    // clients. This function can be called either in loolwsd or loolkit processes, even if only
-    // loolwsd obviously has contact with the actual clients; in loolkit it will be forwarded to
-    // loolwsd for redistribution. (This function must be implemented separately in each program
-    // that uses it, it is not in Util.cpp.)
-    void alertAllUsers(const std::string& cmd, const std::string& kind);
-#else
-    // No-op implementation in the test programs
-    inline void alertAllUsers(const std::string&, const std::string&)
-    {
-    }
-#endif
-
     // Add the file system that 'path' is located on to a list of file systems that are periodically
     // checked for available space. The list is initially empty.
     void registerFileSystemForDiskSpaceChecks(const std::string& path);


More information about the Libreoffice-commits mailing list