update comments

src/kademlia/refresh.cpp

@@ -90,13 +90,6 @@ char const* bootstrap::name() const { return "bootstrap"; }
 
 void bootstrap::done()
 {
-	// TODO: 4 when bootstrapping against our own IP completes,
-	// continue to issue another bootstrap against the deepest,
-	// non-full bucket. when it completes, issue a bootstrap against
-	// one bucket above it, and so on until the bootstrap lookup
-	// against the top level bucket (bucket 0) completes. That's
-	// when the bootstrap is done
-
 #ifdef TORRENT_DHT_VERBOSE_LOGGING
 	TORRENT_LOG(traversal) << "[" << this << "]"
 		<< " bootstrap done, pinging remaining nodes";

src/peer_connection.cpp

@@ -4838,8 +4838,10 @@ namespace libtorrent
 		// the peer and disconnect it
 		bool may_timeout = (m_channel_state[download_channel] & peer_info::bw_network) != 0;
 
-		// TODO: 4 use a deadline_timer for timeouts. Don't rely on second_tick()!
-		// Hook this up to connect timeout as well
+		// TODO: 2 use a deadline_timer for timeouts. Don't rely on second_tick()!
+		// Hook this up to connect timeout as well. This would improve performance
+		// because of less work in second_tick(), and might let use remove ticking
+		// entirely eventually
 		if (may_timeout && d > seconds(timeout()) && !m_connecting && m_reading_bytes == 0
 			&& can_disconnect(error_code(errors::timed_out_inactivity, get_libtorrent_category())))
 		{

src/torrent.cpp

@@ -9622,6 +9622,9 @@ namespace libtorrent
 			state_updated();
 
 		// TODO: 4 this logic doesn't work for seeding torrents that are not ticked
+		// once a torrent is inactive, it may not be ticked anymore, which may
+		// prevent it from ever have m_inactive be set, because there's a delay
+		// deom becoming inactive and trigger auto-manage.
 
 		// this section determines whether the torrent is active or not. When it
 		// changes state, it may also trigger the auto-manage logic to reconsider