got rid of const_cast in CGateKeeper with a mutable mutex

reflector/Buffer.cpp

@@ -176,7 +176,7 @@ CBuffer::operator const char *() const
 
 void CBuffer::DebugDump(std::ofstream &debugout) const
 {
-	// dump an hex line
+	// dump a hex line
 	for ( unsigned int i = 0; i < m_data.size(); i++ )
 	{
 		char sz[16];

reflector/CallsignList.h

@@ -33,8 +33,8 @@ public:
 	CCallsignList();
 
 	// locks
-	void Lock(void)                        { m_Mutex.lock(); }
-	void Unlock(void)                      { m_Mutex.unlock(); }
+	void Lock(void)   const { m_Mutex.lock();   }
+	void Unlock(void) const { m_Mutex.unlock(); }
 
 	// file io
 	virtual bool LoadFromFile(const std::string &str);
@@ -60,8 +60,8 @@ protected:
 	char *TrimWhiteSpaces(char *);
 
 	// data
-	std::mutex      m_Mutex;
-	std::string     m_Filename;
-	time_t          m_LastModTime;
+	mutable std::mutex m_Mutex;
+	std::string        m_Filename;
+	time_t             m_LastModTime;
 	std::list<CCallsignListItem> m_Callsigns;
 };

reflector/CodecStream.cpp

@@ -133,10 +133,6 @@ void CCodecStream::Task(void)
 		}
 		m_RTSum += rt;
 		m_RTCount++;
-#ifdef DEBUG
-		if (0 == m_RTCount % 50)
-			ReportStats();
-#endif
 
 		if ( m_LocalQueue.IsEmpty() )
 		{

reflector/GateKeeper.cpp

@@ -204,17 +204,20 @@ bool CGateKeeper::IsNodeListedOk(const CCallsign &callsign, const CIp &ip, char
 	{
 		// first check if callsign is in white list
 		// note if white list is empty, everybody is authorized
-		const_cast<CCallsignList &>(m_NodeWhiteList).Lock();
+		m_NodeWhiteList.Lock();
 		if ( !m_NodeWhiteList.empty() )
 		{
 			ok = m_NodeWhiteList.IsCallsignListedWithWildcard(callsign, module);
 		}
-		const_cast<CCallsignList &>(m_NodeWhiteList).Unlock();
+		m_NodeWhiteList.Unlock();
 
 		// then check if not blacklisted
-		const_cast<CCallsignList &>(m_NodeBlackList).Lock();
-		ok &= !m_NodeBlackList.IsCallsignListedWithWildcard(callsign);
-		const_cast<CCallsignList &>(m_NodeBlackList).Unlock();
+		if (ok)
+		{
+			m_NodeBlackList.Lock();
+			ok = !m_NodeBlackList.IsCallsignListedWithWildcard(callsign);
+			m_NodeBlackList.Unlock();
+		}
 	}
 
 	// done
@@ -232,12 +235,12 @@ bool CGateKeeper::IsPeerListedOk(const CCallsign &callsign, const CIp &ip, char
 	if ( ok )
 	{
 		// look for an exact match in the list
-		const_cast<CPeerCallsignList &>(m_PeerList).Lock();
+		m_PeerList.Lock();
 		if ( !m_PeerList.empty() )
 		{
 			ok = m_PeerList.IsCallsignListed(callsign, module);
 		}
-		const_cast<CPeerCallsignList &>(m_PeerList).Unlock();
+		m_PeerList.Unlock();
 	}
 
 	// done
@@ -254,12 +257,12 @@ bool CGateKeeper::IsPeerListedOk(const CCallsign &callsign, const CIp &ip, char
 	if ( ok )
 	{
 		// look for an exact match in the list
-		const_cast<CPeerCallsignList &>(m_PeerList).Lock();
+		m_PeerList.Lock();
 		if ( !m_PeerList.empty() )
 		{
 			ok = m_PeerList.IsCallsignListed(callsign, modules);
 		}
-		const_cast<CPeerCallsignList &>(m_PeerList).Unlock();
+		m_PeerList.Unlock();
 	}
 
 	// done