Skip to content

Commit

Permalink
chore: Fix wording in comments (#2411)
Browse files Browse the repository at this point in the history
  • Loading branch information
larseggert authored Feb 3, 2025
1 parent e365730 commit 3d29b57
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion neqo-transport/src/connection/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2502,7 +2502,7 @@ impl Connection {
// If the client has more CRYPTO data queued up, do not coalesce if
// this packet is an Initial. Without this, 0-RTT packets could be
// coalesced with the first Initial, which some server (e.g., ours)
// do not support, because may do not save packets they can't
// do not support, because they may not save packets they can't
// decrypt yet.
if self.role == Role::Client
&& *space == PacketNumberSpace::Initial
Expand Down
4 changes: 2 additions & 2 deletions neqo-transport/src/connection/tests/handshake.rs
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ fn send_05rtt() {
/// Test that a client buffers 0.5-RTT data when it arrives early.
#[test]
fn reorder_05rtt() {
// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
let mut client = new_client(ConnectionParameters::default().mlkem(false));
let mut server = default_server();

Expand Down Expand Up @@ -392,7 +392,7 @@ fn reorder_05rtt_with_0rtt() {
client.process_input(ticket, now);

let token = get_tokens(&mut client).pop().unwrap();
// This tests makes too many assumptions about what's in the packets to work with multi-packet
// This test makes too many assumptions about what's in the packets to work with multi-packet
// MLKEM flights.
let mut client = new_client(ConnectionParameters::default().mlkem(false));
client.enable_resumption(now, token).unwrap();
Expand Down
2 changes: 1 addition & 1 deletion neqo-transport/src/connection/tests/idle.rs
Original file line number Diff line number Diff line change
Expand Up @@ -683,7 +683,7 @@ fn keep_alive_with_ack_eliciting_packet_lost() {
// - Idle time out will trigger (at the timeout + IDLE_TIMEOUT)
const IDLE_TIMEOUT: Duration = Duration::from_millis(6000);

// This tests makes too many assumptions about single-packet flights and PTOs for multi-packet
// This test makes too many assumptions about single-packet flights and PTOs for multi-packet
// MLKEM flights to work.
let mut client = new_client(
ConnectionParameters::default()
Expand Down
10 changes: 5 additions & 5 deletions neqo-transport/src/connection/tests/recovery.rs
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ fn pto_initial() {
const INITIAL_PTO: Duration = Duration::from_millis(300);
let mut now = now();

// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
qdebug!("---- client: generate CH");
let mut client = new_client(ConnectionParameters::default().mlkem(false));
let pkt1 = client.process_output(now).dgram();
Expand Down Expand Up @@ -397,7 +397,7 @@ fn pto_handshake_frames() {
fn handshake_ack_pto() {
const RTT: Duration = Duration::from_millis(10);
let mut now = now();
// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// to work.
let mut client = new_client(ConnectionParameters::default().mlkem(false));
let mut server = default_server();
Expand Down Expand Up @@ -581,7 +581,7 @@ fn lost_but_kept_and_lr_timer() {
fn loss_time_past_largest_acked() {
const RTT: Duration = Duration::from_secs(10);
const INCR: Duration = Duration::from_millis(1);
// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// to work.
let mut client = new_client(ConnectionParameters::default().mlkem(false));
let mut server = default_server();
Expand Down Expand Up @@ -760,7 +760,7 @@ fn expected_pto(rtt: Duration) -> Duration {

#[test]
fn fast_pto() {
// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// to work.
let mut client = new_client(
ConnectionParameters::default()
Expand Down Expand Up @@ -803,7 +803,7 @@ fn fast_pto() {
/// based on the "true" value of the timer.
#[test]
fn fast_pto_persistent_congestion() {
// This tests makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// This test makes too many assumptions about single-packet PTOs for multi-packet MLKEM flights
// to work.
let mut client = new_client(
ConnectionParameters::default()
Expand Down

0 comments on commit 3d29b57

Please sign in to comment.