refactor: don't route startNode through AppUtil

src/lib/deskflow/AppUtil.h

@@ -35,7 +35,6 @@ public:
 
   // Virtual Methods subclasses can impliment
   virtual int run() = 0;
-  virtual void startNode() = 0;
   virtual std::vector<std::string> getKeyboardLayoutList() = 0;
   virtual std::string getCurrentLanguageCode() = 0;
 

src/lib/deskflow/ClientApp.cpp

@@ -323,7 +323,7 @@ int ClientApp::mainLoop()
   setSocketMultiplexer(std::make_unique<SocketMultiplexer>());
 
   // start client, etc
-  appUtil().startNode();
+  startNode();
 
   // run event loop.  if startClient() failed we're supposed to retry
   // later.  the timer installed by startClient() will take care of

src/lib/deskflow/ServerApp.cpp

@@ -517,7 +517,7 @@ int ServerApp::mainLoop()
 
   // start server, etc
   try {
-    appUtil().startNode();
+    startNode();
   } catch (...) {
     // a fatal startup failure exits by throwing, skipping the cleanup below;
     // tear down here so the screen's worker threads don't outlive the app.

src/lib/deskflow/unix/AppUtilUnix.cpp

@@ -34,11 +34,6 @@ int AppUtilUnix::run()
   return app().runInner(&startStatic);
 }
 
-void AppUtilUnix::startNode()
-{
-  app().startNode();
-}
-
 std::vector<std::string> AppUtilUnix::getKeyboardLayoutList()
 {
   std::vector<std::string> layoutLangCodes;

src/lib/deskflow/unix/AppUtilUnix.h

@@ -20,7 +20,6 @@ public:
   ~AppUtilUnix() override = default;
 
   int run() override;
-  void startNode() override;
   std::vector<std::string> getKeyboardLayoutList() override;
   std::string getCurrentLanguageCode() override;
 };

src/lib/deskflow/win32/AppUtilWindows.cpp

@@ -126,11 +126,6 @@ AppUtilWindows &AppUtilWindows::instance()
   return (AppUtilWindows &)AppUtil::instance();
 }
 
-void AppUtilWindows::startNode()
-{
-  app().startNode();
-}
-
 std::vector<std::string> AppUtilWindows::getKeyboardLayoutList()
 {
   std::vector<std::string> layoutLangCodes;

src/lib/deskflow/win32/AppUtilWindows.h

@@ -38,7 +38,6 @@ public:
   int daemonNTMainLoop();
   int run() override;
   void exitApp(int code) override;
-  void startNode() override;
   std::vector<std::string> getKeyboardLayoutList() override;
   std::string getCurrentLanguageCode() override;
   HKL getCurrentKeyboardLayout() const;

src/unittests/client/ServerProxyTests.cpp

@@ -35,10 +35,6 @@ public:
     return 0;
   }
 
-  void startNode() override
-  {
-  }
-
   std::vector<std::string> getKeyboardLayoutList() override
   {
     return {"en"};

src/unittests/platform/EiKeyStateTests.cpp

@@ -25,10 +25,6 @@ public:
     return 0;
   }
 
-  void startNode() override
-  {
-  }
-
   std::vector<std::string> getKeyboardLayoutList() override
   {
     return {"en"};