Packets unsent due to the send-buffer being filled should no longer count as lost traffic.

Version tentatively bumped to 0.1.8.

Author: Neil Tallim

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rperf"
-version = "0.1.7"
+version = "0.1.8"
 description = "validates network throughput capacity and reliability"
 authors = ["Neil Tallim <neiltallim@3d-p.com>"]
 edition = "2021"

src/stream/udp.rs

@@ -303,8 +303,8 @@ pub mod receiver {
                 self.process_jitter(&source_timestamp, &mut history);
 
                 if history.unbroken_sequence > history.longest_unbroken_sequence {
-                    history.longest_unbroken_sequence = history.unbroken_sequence
-                    history.longest_jitter_seconds = history.jitter_seconds
+                    history.longest_unbroken_sequence = history.unbroken_sequence;
+                    history.longest_jitter_seconds = history.jitter_seconds;
                 }
             } else {
                 history.unbroken_sequence = 0;
@@ -561,6 +561,8 @@ pub mod sender {
                     Err(e) if e.kind() == std::io::ErrorKind::WouldBlock => { //send-buffer is full
                         //nothing to do, but avoid burning CPU cycles
                         sleep(BUFFER_FULL_TIMEOUT);
+                        //roll back the packet-ID because nothing was actually emitted
+                        self.next_packet_id -= 1;
                     },
                     Err(e) => {
                         return Some(Err(Box::new(e)));