Remove useless TODO comments (#350)

Tencent · Aug 1, 2024 · 8f4a73a · 8f4a73a
1 parent 2cfd71c
commit 8f4a73a
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 8 deletions.
diff --git a/src/connection/connection.rs b/src/connection/connection.rs
@@ -693,8 +693,6 @@ impl Connection {
 
         self.flags.insert(NeedSendAckEliciting);
 
-        // TODO: notify pacer on the path
-
         Ok(read)
     }
 
@@ -1468,7 +1466,7 @@ impl Connection {
         let info = PacketInfo {
             src: path.local_addr(),
             dst: path.remote_addr(),
-            time: time::Instant::now(), // TODO: Set an appropriate sending time
+            time: time::Instant::now(),
         };
         Ok((done, info))
     }
@@ -3177,7 +3175,7 @@ impl Connection {
         // current Probe Timeout (PTO).
         // See RFC 9000 Section 10.1
         let path_pto = match self.paths.get_active_mut() {
-            Ok(p) => p.recovery.rtt.pto_base(), // TODO: revisit
+            Ok(p) => p.recovery.rtt.pto_base(),
             Err(_) => time::Duration::ZERO,
         };
         let idle_timeout = cmp::max(idle_timeout, 3 * path_pto);

diff --git a/src/connection/recovery.rs b/src/connection/recovery.rs
@@ -165,7 +165,7 @@ impl Recovery {
         let ack_eliciting = pkt.ack_eliciting;
         let sent_size = pkt.sent_size;
 
-        pkt.time_sent = now; // TODO: REVISIT
+        pkt.time_sent = now;
         let space = match spaces.get_mut(space_id) {
             Some(space) => space,
             None => return,
@@ -286,7 +286,7 @@ impl Recovery {
             self.congestion.end_ack();
         }
 
-        self.pto_count = 0; // TODO: REVISIT
+        self.pto_count = 0;
         self.set_loss_detection_timer(space_id, spaces, handshake_status, now);
         Ok((lost_packets, lost_bytes))
     }

diff --git a/src/lib.rs b/src/lib.rs
@@ -372,7 +372,6 @@ impl Config {
     /// # Ok::<(), tquic::error::Error>(())
     /// ```
     pub fn new() -> Result<Self> {
-        // TODO: review default value
         let local_transport_params = TransportParams {
             initial_max_data: 10485760,
             initial_max_stream_data_bidi_local: 5242880,

diff --git a/src/tls/tls.rs b/src/tls/tls.rs
@@ -221,7 +221,6 @@ pub(crate) struct DefaultTlsConfigSelector {
 }
 
 impl TlsConfigSelector for DefaultTlsConfigSelector {
-    // TODO: support local and peer address.
     /// Get default TLS config.
     fn get_default(&self) -> Option<Arc<TlsConfig>> {
         Some(self.tls_config.clone())