[Libreoffice-commits] online.git: wsd/DocumentBroker.hpp wsd/LOOLWSD.hpp
Tor Lillqvist (via logerrit)
logerrit at kemper.freedesktop.org
Sat Apr 18 14:35:08 UTC 2020
wsd/DocumentBroker.hpp | 6 ++++--
wsd/LOOLWSD.hpp | 8 ++++----
2 files changed, 8 insertions(+), 6 deletions(-)
New commits:
commit 7a416cbf0a3cf6724d5ff21128ed46cb82c42a1a
Author: Tor Lillqvist <tml at collabora.com>
AuthorDate: Sat Apr 18 16:44:50 2020 +0300
Commit: Tor Lillqvist <tml at collabora.com>
CommitDate: Sat Apr 18 16:34:49 2020 +0200
Clarify what ChildProcess and WSProcess are
Change-Id: I13dd37c840d2c8b811bb56614fbc9f8ee2e44d40
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92475
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
Reviewed-by: Tor Lillqvist <tml at collabora.com>
diff --git a/wsd/DocumentBroker.hpp b/wsd/DocumentBroker.hpp
index d161be4aa..99391c0d0 100644
--- a/wsd/DocumentBroker.hpp
+++ b/wsd/DocumentBroker.hpp
@@ -58,8 +58,10 @@ public:
#include "LOOLWSD.hpp"
-/// Represents a new LOK child that is read
-/// to host a document.
+/// A ChildProcess object represents a KIT process that hosts a document and manipulates the
+/// document using the LibreOfficeKit API. It isn't actually a child of the WSD process, but a
+/// grandchild. The comments loosely talk about "child" anyway.
+
class ChildProcess : public WSProcess
{
public:
diff --git a/wsd/LOOLWSD.hpp b/wsd/LOOLWSD.hpp
index 368b58076..c391cc055 100644
--- a/wsd/LOOLWSD.hpp
+++ b/wsd/LOOLWSD.hpp
@@ -38,10 +38,10 @@ std::shared_ptr<ChildProcess> getNewChild_Blocks(
const std::string& uri
#endif
);
-// This is common code used to setup as socket to both
-// forkit and child document processes via a websocket.
-// In general, a WSProcess instance represents a child
-// process with which we can communicate through websocket.
+// A WSProcess object in the WSD process represents a descendant process, either the direct child
+// process FORKIT or a grandchild KIT process, with which the WSD process communicates through a
+// WebSocket.
+
class WSProcess
{
public:
More information about the Libreoffice-commits
mailing list