fix another format string issue and add comments to kademlia

include/libtorrent/kademlia/node.hpp

@@ -197,6 +197,7 @@ protected:
 	~udp_socket_interface() {}
 };
 
+// TODO: 3 rename this to just node
 class TORRENT_EXTRA_EXPORT node_impl : boost::noncopyable
 {
 typedef std::map<node_id, torrent_entry> table_t;

src/kademlia/dht_tracker.cpp

@@ -139,6 +139,8 @@ namespace libtorrent { namespace dht
 
 	// class that puts the networking and the kademlia node in a single
 	// unit and connecting them together.
+	// TODO: 3 don't pass in session here. pass in dht_observer and maybe add
+	// a way to ask it for the external IP. use bdecode_node instead of entry
 	dht_tracker::dht_tracker(libtorrent::aux::session_impl& ses
 		, rate_limited_udp_socket& sock
 		, dht_settings const& settings, counters& cnt, entry const* state)

src/session_impl.cpp

@@ -2391,7 +2391,7 @@ retry:
 		{
 #ifndef TORRENT_DISABLE_LOGGING
 			session_log("%s <== INCOMING CONNECTION FAILED, could "
-				"not retrieve remote endpoint %s: %s"
+				"not retrieve remote endpoint: %s"
 				, print_endpoint(endp).c_str(), ec.message().c_str());
 #endif
 			return;
@@ -5400,6 +5400,7 @@ retry:
 		}
 	}
 
+	// TODO: 3 use bdecode_node instead of entry.
 	void session_impl::start_dht(entry const& startup_state)
 	{
 		INVARIANT_CHECK;