Document setting once would suffice

mozilla · May 2, 2024 · a27b9bc · a27b9bc
1 parent f5f1bb0
commit a27b9bc
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/neqo-http3/src/send_message.rs b/neqo-http3/src/send_message.rs
@@ -179,6 +179,9 @@ impl SendStream for SendMessage {
             .stream_avail_send_space(self.stream_id())
             .map_err(|e| Error::map_stream_send_errors(&e.into()))?;
         if available < MIN_DATA_FRAME_SIZE {
+            // Setting this once, instead of every time the available send space
+            // is exhausted, would suffice. That said, function call should be
+            // cheap, thus not worth optimizing.
             conn.stream_set_writable_event_low_watermark(
                 self.stream_id(),
                 NonZeroUsize::new(MIN_DATA_FRAME_SIZE).unwrap(),