class WQueue: method 'T remove(const long delay)' removed.

lib/wqueue.h

@@ -21,10 +21,7 @@
 #define WQUEUE_H_
 
 #include <list>
-#include <ctime>
 #include <pthread.h>
-#include <sys/time.h>
-#include <errno.h>
 
 template <typename T> class WQueue
 {
@@ -67,34 +64,6 @@ public:
 		return item;
 	}
 
-	T remove(const long delay)
-	{
-		struct timeval tv;
-		struct timezone tz;
-		struct timespec timeout;
-
-		int ret = 0;
-
-		pthread_mutex_lock(&m_mutex);
-
-		gettimeofday(&tv, &tz);
-		timeout.tv_sec = tv.tv_sec + delay;
-		timeout.tv_nsec = tv.tv_usec * 1000;
-
-		while (m_queue.size() == 0 && ret != ETIMEDOUT)
-			ret = pthread_cond_timedwait(&m_condv, &m_mutex, &timeout);
-
-		if (ret == ETIMEDOUT)
-			return NULL;
-
-		T item = m_queue.front();
-		m_queue.pop_front();
-
-		pthread_mutex_unlock(&m_mutex);
-
-		return item;
-	}
-
 	int size()
 	{
 		pthread_mutex_lock(&m_mutex);

src/baseloop.cpp

@@ -96,7 +96,6 @@ std::string BaseLoop::decodeMessage(const std::string& data)
 			m_ebusloop->addBusCommand(new BusCommand(type, ebusCommand));
 			BusCommand* busCommand = m_ebusloop->getBusCommand();
 
-			if (busCommand != NULL) {
 			if (busCommand->getResult().c_str()[0] != '-') {
 				// decode data
 				Command* command = new Command(index, (*m_commands)[index], busCommand->getResult());
@@ -112,10 +111,6 @@ std::string BaseLoop::decodeMessage(const std::string& data)
 
 
 			delete busCommand;
-			} else {
-				L.log(bas, error, " -7: receive timeout");
-				result << "-7: receive timeout";
-			}
 
 		} else {
 			result << "ebus command not found";
@@ -155,7 +150,6 @@ std::string BaseLoop::decodeMessage(const std::string& data)
 			m_ebusloop->addBusCommand(new BusCommand(type, ebusCommand));
 			BusCommand* busCommand = m_ebusloop->getBusCommand();
 
-			if (busCommand != NULL) {
 			if (busCommand->getResult().c_str()[0] != '-') {
 				// decode result
 				if (busCommand->getResult().substr(busCommand->getResult().length()-8) == "00000000")
@@ -169,11 +163,6 @@ std::string BaseLoop::decodeMessage(const std::string& data)
 			}
 
 			delete busCommand;
-			} else {
-				L.log(bas, error, " -7: receive timeout");
-				result << "-7: receive timeout";
-			}
-
 			delete command;
 
 		} else {
@@ -231,17 +220,12 @@ std::string BaseLoop::decodeMessage(const std::string& data)
 			m_ebusloop->addBusCommand(new BusCommand(type, ebusCommand));
 			BusCommand* busCommand = m_ebusloop->getBusCommand();
 
-			if (busCommand != NULL) {
 			if (busCommand->getResult().c_str()[0] == '-')
 				L.log(bas, error, " %s", busCommand->getResult().c_str());
 
 			result << busCommand->getResult();
 
 			delete busCommand;
-			} else {
-				L.log(bas, error, " -7: receive timeout");
-				result << "-7: receive timeout";
-			}
 
 		} else {
 			result << "specified message type is incorrect";

src/ebusloop.h

@@ -40,7 +40,7 @@ public:
 	std::string getData() { return m_cycBuffer.remove(); }
 
 	void addBusCommand(BusCommand* busCommand) { m_sendBuffer.add(busCommand); }
-	BusCommand* getBusCommand() { return m_recvBuffer.remove(10); }
+	BusCommand* getBusCommand() { return m_recvBuffer.remove(); }
 
 	void dump(const bool dumpState) { m_bus->setDumpState(dumpState); }