From e227e53d7b7afb63255ae9df4cdc0ef56ff577d9 Mon Sep 17 00:00:00 2001
From: Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com>
Date: Tue, 9 Feb 2010 12:20:20 +0000
Subject: 2010-02-09  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>

	Wrapped logger->debug(...) with if(logger->debug()) { ... }
	* src/AbstractCommand.cc
	* src/AdaptiveURISelector.cc
	* src/BtPieceMessage.cc
	* src/BtPortMessage.cc
	* src/CreateRequestCommand.cc
	* src/DHTAbstractNodeLookupTask.cc
	* src/DHTBucket.cc
	* src/DHTGetPeersCommand.cc
	* src/DHTMessageDispatcherImpl.cc
	* src/DHTMessageFactoryImpl.cc
	* src/DHTMessageTracker.cc
	* src/DHTPeerAnnounceStorage.cc
	* src/DHTRoutingTable.cc
	* src/DHTSetup.cc
	* src/DefaultBtAnnounce.cc
	* src/DefaultBtInteractive.cc
	* src/DefaultBtMessageDispatcher.cc
	* src/DefaultBtMessageReceiver.cc
	* src/DefaultBtProgressInfoFile.cc
	* src/DefaultBtRequestFactory.cc
	* src/DefaultPeerStorage.cc
	* src/DefaultPieceStorage.cc
	* src/DownloadCommand.cc
	* src/DownloadEngine.cc
	* src/EpollEventPoll.cc
	* src/FileAllocationCommand.cc
	* src/FileEntry.cc
	* src/FtpNegotiationCommand.cc
	* src/HandshakeExtensionMessage.cc
	* src/HttpListenCommand.cc
	* src/HttpServer.cc
	* src/HttpSkipResponseCommand.cc
	* src/IteratableChunkChecksumValidator.cc
	* src/Logger.h
	* src/MSEHandshake.cc
	* src/MetalinkPostDownloadHandler.cc
	* src/MultiDiskAdaptor.cc
	* src/PeerAbstractCommand.cc
	* src/PeerConnection.cc
	* src/PeerListenCommand.cc
	* src/PeerReceiveHandshakeCommand.cc
	* src/RequestGroup.cc
	* src/RequestGroupMan.cc
	* src/SegmentMan.cc
	* src/SelectEventPoll.cc
	* src/ServerStat.cc
	* src/SimpleBtMessage.cc
	* src/SocketCore.cc
	* src/TrackerWatcherCommand.cc
	* src/UTMetadataDataExtensionMessage.cc
	* src/UTMetadataRequestFactory.cc
	* src/UTMetadataRequestTracker.cc
	* src/XmlRpcMethod.cc
---
 src/DHTAbstractNodeLookupTask.cc | 41 ++++++++++++++++++++++++++--------------
 1 file changed, 27 insertions(+), 14 deletions(-)

(limited to 'src/DHTAbstractNodeLookupTask.cc')

diff --git a/src/DHTAbstractNodeLookupTask.cc b/src/DHTAbstractNodeLookupTask.cc
index 459074ce..f828c4dc 100644
--- a/src/DHTAbstractNodeLookupTask.cc
+++ b/src/DHTAbstractNodeLookupTask.cc
@@ -71,16 +71,21 @@ void DHTAbstractNodeLookupTask::onReceived(const SharedHandle<DHTMessage>& messa
     if(memcmp(_localNode->getID(), (*i)->_node->getID(), DHT_ID_LENGTH) != 0) {
       _entries.push_front(*i);
       ++count;
-      _logger->debug("Received nodes: id=%s, ip=%s",
-                     util::toHex((*i)->_node->getID(), DHT_ID_LENGTH).c_str(),
-                     (*i)->_node->getIPAddress().c_str());
+      if(_logger->debug()) {
+        _logger->debug("Received nodes: id=%s, ip=%s",
+                       util::toHex((*i)->_node->getID(), DHT_ID_LENGTH).c_str(),
+                       (*i)->_node->getIPAddress().c_str());
+      }
     }
   }
-
-  _logger->debug("%u node lookup entries added.", count);
+  if(_logger->debug()) {
+    _logger->debug("%u node lookup entries added.", count);
+  }
   std::stable_sort(_entries.begin(), _entries.end(), DHTIDCloser(_targetID));
   _entries.erase(std::unique(_entries.begin(), _entries.end()), _entries.end());
-  _logger->debug("%u node lookup entries are unique.", _entries.size());
+  if(_logger->debug()) {
+    _logger->debug("%u node lookup entries are unique.", _entries.size());
+  }
   if(_entries.size() > DHTBucket::K) {
     _entries.erase(_entries.begin()+DHTBucket::K, _entries.end());
   }
@@ -89,8 +94,10 @@ void DHTAbstractNodeLookupTask::onReceived(const SharedHandle<DHTMessage>& messa
 
 void DHTAbstractNodeLookupTask::onTimeout(const SharedHandle<DHTNode>& node)
 {
-  _logger->debug("node lookup message timeout for node ID=%s",
-                 util::toHex(node->getID(), DHT_ID_LENGTH).c_str());
+  if(_logger->debug()) {
+    _logger->debug("node lookup message timeout for node ID=%s",
+                   util::toHex(node->getID(), DHT_ID_LENGTH).c_str());
+  }
   --_inFlightMessage;
   for(std::deque<SharedHandle<DHTNodeLookupEntry> >::iterator i = _entries.begin(); i != _entries.end(); ++i) {
     if((*i)->_node == node) {
@@ -107,15 +114,19 @@ void DHTAbstractNodeLookupTask::sendMessageAndCheckFinish()
     sendMessage();
   }
   if(_inFlightMessage == 0) {
-    _logger->debug("Finished node_lookup for node ID %s",
-                   util::toHex(_targetID, DHT_ID_LENGTH).c_str());
+    if(_logger->debug()) {
+      _logger->debug("Finished node_lookup for node ID %s",
+                     util::toHex(_targetID, DHT_ID_LENGTH).c_str());
+    }
     onFinish();
     updateBucket();
     _finished = true;
   } else {
-    _logger->debug("%d in flight message for node ID %s",
-                   _inFlightMessage,
-                   util::toHex(_targetID, DHT_ID_LENGTH).c_str());
+    if(_logger->debug()) {
+      _logger->debug("%d in flight message for node ID %s",
+                     _inFlightMessage,
+                     util::toHex(_targetID, DHT_ID_LENGTH).c_str());
+    }
   }
 }
 
@@ -151,7 +162,9 @@ void DHTAbstractNodeLookupTask::startup()
     _inFlightMessage = 0;
     sendMessage();
     if(_inFlightMessage == 0) {
-      _logger->debug("No message was sent in this lookup stage. Finished.");
+      if(_logger->debug()) {
+        _logger->debug("No message was sent in this lookup stage. Finished.");
+      }
       _finished = true;
     }
   }
-- 
cgit