* remove dispensable method of CAmTelnetMenuHelper
authorFrank Herchet <frank.fh.herchet@bmw.de>
Thu, 23 Feb 2012 09:32:26 +0000 (10:32 +0100)
committerFrank Herchet <frank.fh.herchet@bmw.de>
Thu, 23 Feb 2012 09:32:26 +0000 (10:32 +0100)
AudioManagerDaemon/include/CAmTelnetMenuHelper.h
AudioManagerDaemon/src/CAmTelnetMenuHelper.cpp
AudioManagerDaemon/src/TelnetServer.cpp

index 73d302a..5322526 100644 (file)
@@ -66,8 +66,6 @@ public:
 
     ~CAmTelnetMenuHelper();
 
-    void setTelnetServer(TelnetServer* iTelnetServer);
-
     void newSocketConnection(int filedescriptor);
 
     void socketConnectionsClosed(int filedescriptor);
index a3180f3..2c9ea18 100644 (file)
@@ -113,13 +113,6 @@ void CAmTelnetMenuHelper::createCommandMaps()
 }
 
 /****************************************************************************/
-void CAmTelnetMenuHelper::setTelnetServer(TelnetServer* iTelnetServer)
-/****************************************************************************/
-{
-    mpTelenetServer = iTelnetServer;
-}
-
-/****************************************************************************/
 void CAmTelnetMenuHelper::newSocketConnection(int filedescriptor)
 /****************************************************************************/
 {
@@ -275,7 +268,7 @@ void CAmTelnetMenuHelper::sendCurrentCmdPrompt(int & filedescriptor)
     }
     else
     {
-        logError("[TN] sendCurrentCmdPrompt, fd not found",filedescriptor);
+        logInfo("[TN] sendCurrentCmdPrompt, fd not found: ",filedescriptor);
     }
 }
 
@@ -349,10 +342,9 @@ void CAmTelnetMenuHelper::exitCommandExec(std::queue<std::string> & CmdQueue, in
     it = mCurrentMainStateMap.find(filedescriptor);
     if (it != mCurrentMainStateMap.end())
     {
-        logInfo("[TN] exitCommandExec, mpTelenetServer == NULL, fd ",filedescriptor);
-
         if (NULL != mpTelenetServer)
         {
+            logInfo("[TN] exitCommandExec, removing fd ",filedescriptor);
             mpTelenetServer->disconnectClient(filedescriptor);
             mCurrentMainStateMap.erase(it);
         }
index 06c75b7..1cae985 100644 (file)
@@ -103,7 +103,7 @@ TelnetServer::TelnetServer(SocketHandler *iSocketHandler, CommandSender *iComman
         logInfo("TelnetServer::TelnetServer started listening on port", mServerPort);
 
     int a=1;
-    ioctl (mConnectFD, FIONBIO, (char *) &a); // should we use the posix call fcntl(mConnectFD, F_SETFL, O_NONBLOCK)
+    ioctl (mConnectFD, FIONBIO, (char *) &a);
     setsockopt (mConnectFD, SOL_SOCKET, SO_KEEPALIVE, (char *) &a, sizeof (a));
 
     short events = 0;
@@ -113,7 +113,6 @@ TelnetServer::TelnetServer(SocketHandler *iSocketHandler, CommandSender *iComman
 
 TelnetServer::~TelnetServer()
 {
-   mTelnetMenuHelper.setTelnetServer(NULL);
 }
 
 void TelnetServer::connectSocket(const pollfd pfd, const sh_pollHandle_t handle, void *userData)