Make SDesktop::start()/stop() optional

Let's avoid requring these as a desktop implementation can now set up
everything in the init() method.

1 files changed, 2 insertions, 8 deletions

diff --git a/common/rfb/SDesktop.h b/common/rfb/SDesktop.h
index e3871616..560ee7ff 100644
--- a/common/rfb/SDesktop.h
+++ b/common/rfb/SDesktop.h
@@ -58,14 +58,12 @@ namespace rfb {
     // needed when there are no clients.  A valid PixelBuffer must have been
     // set via the VNCServer's setPixelBuffer() method by the time this call
     // returns.
-
-    virtual void start() = 0;
+    virtual void start() {}
 
     // stop() is called by the server when there are no longer any
     // authenticated clients, and therefore the desktop can cease any
     // expensive tasks.
-
-    virtual void stop() = 0;
+    virtual void stop() {}
 
     // queryConnection() is called when a connection has been
     // successfully authenticated.  The sock and userName arguments
@@ -143,10 +141,6 @@ namespace rfb {
       server = vs;
       server->setPixelBuffer(buffer);
     }
-    virtual void start() {
-    }
-    virtual void stop() {
-    }
     virtual void queryConnection(network::Socket* sock,
                                  const char* /*userName*/) {
       server->approveConnection(sock, true, NULL);