From 7e1093e976dbadfebcce893b943f6b52e1d4f8d5 Mon Sep 17 00:00:00 2001
From: Pierre Ossman <ossman@cendio.se>
Date: Thu, 11 Oct 2018 08:08:19 +0200
Subject: [PATCH] Clean up headers

Add missing comments and clearly separate methods from attributes.
---
 common/rfb/VNCSConnectionST.h |  4 ++++
 common/rfb/VNCServerST.h      | 23 +++++++++++++----------
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/common/rfb/VNCSConnectionST.h b/common/rfb/VNCSConnectionST.h
index d09f733c..1b7d59ac 100644
--- a/common/rfb/VNCSConnectionST.h
+++ b/common/rfb/VNCSConnectionST.h
@@ -100,6 +100,9 @@ namespace rfb {
     bool needRenderedCursor();
 
     network::Socket* getSock() { return sock; }
+
+    // Change tracking
+
     void add_changed(const Region& region) { updates.add_changed(region); }
     void add_copied(const Region& dest, const Point& delta) {
       updates.add_copied(dest, delta);
@@ -152,6 +155,7 @@ namespace rfb {
     void setLEDState(unsigned int state);
     void setSocketTimeouts();
 
+  private:
     network::Socket* sock;
     CharArray peerEndpoint;
     bool reverseConnection;
diff --git a/common/rfb/VNCServerST.h b/common/rfb/VNCServerST.h
index f4b0d3fc..545b9a49 100644
--- a/common/rfb/VNCServerST.h
+++ b/common/rfb/VNCServerST.h
@@ -148,6 +148,8 @@ namespace rfb {
     // ready to be sent to clients
     Region getPendingRegion();
 
+    // getRenderedCursor() returns an up to date version of the server
+    // side rendered cursor buffer
     const RenderedCursor* getRenderedCursor();
 
   protected:
@@ -160,6 +162,17 @@ namespace rfb {
     void startDesktop();
     void stopDesktop();
 
+    // - Check how many of the clients are authenticated.
+    int authClientCount();
+
+    bool needRenderedCursor();
+    void startFrameClock();
+    void stopFrameClock();
+    void writeUpdate();
+
+    bool getComparerState();
+
+  protected:
     Blacklist blacklist;
     Blacklist* blHosts;
 
@@ -183,16 +196,6 @@ namespace rfb {
     RenderedCursor renderedCursor;
     bool renderedCursorInvalid;
 
-    // - Check how many of the clients are authenticated.
-    int authClientCount();
-
-    bool needRenderedCursor();
-    void startFrameClock();
-    void stopFrameClock();
-    void writeUpdate();
-
-    bool getComparerState();
-
     KeyRemapper* keyRemapper;
 
     time_t lastUserInputTime;
-- 
2.39.5