Skip to content

Commit 10c181f

Browse files
committed
fix bug caused by copy pasta
1 parent 898cd49 commit 10c181f

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

src/bridge_stream.rs

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -79,21 +79,21 @@ fn gather_packets(
7979
let v = std::mem::replace(&mut iface.existing, vec![]);
8080
to_sort.extend(v);
8181
}
82-
println!("Have {} existing packets", to_sort.len());
82+
trace!("Have {} existing packets", to_sort.len());
8383
if let Some(ts) = gather_to {
8484
for iface in interfaces.iter_mut() {
8585
let current = std::mem::replace(&mut iface.current, vec![]);
8686
let t: (Vec<_>, Vec<_>) = current.into_iter().partition(|p| {
8787
*p.timestamp() < ts
8888
});
8989
let (before_ts, after_ts) = t;
90-
println!("Adding {} packets based on timestamp, {} packets adding to existing", before_ts.len(), after_ts.len());
90+
trace!("Adding {} packets based on timestamp, {} packets adding to existing", before_ts.len(), after_ts.len());
9191
to_sort.extend(before_ts);
9292
iface.existing = after_ts;
9393
}
9494
} else {
9595
for iface in interfaces.iter_mut() {
96-
println!("Moving {} packets into existing", iface.current.len());
96+
trace!("Moving {} packets into existing", iface.current.len());
9797
std::mem::swap(&mut iface.existing, &mut iface.current);
9898
}
9999
}
@@ -106,7 +106,7 @@ impl Stream for BridgeStream {
106106

107107
fn poll_next(self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {
108108
let this = self.project();
109-
debug!("Interfaces: {:?}", this.interfaces.len());
109+
trace!("Interfaces: {:?}", this.interfaces.len());
110110
let interfaces: &mut VecDeque<BridgedInterface> = this.interfaces;
111111
let config: &mut Config = this.config; //TODO use the Self {} extractor
112112

@@ -119,7 +119,7 @@ impl Stream for BridgeStream {
119119
}
120120
if let Some(mut existing_delay) = iface.delaying.take() { //Check the interface for a delay..
121121
if let Poll::Pending = Pin::new(&mut existing_delay).poll(cx) { //still delayed?
122-
debug!("Delaying");
122+
trace!("Delaying");
123123
iface.delaying = Some(existing_delay);
124124
continue; // do another iteration on another iface
125125
}
@@ -128,7 +128,7 @@ impl Stream for BridgeStream {
128128
let mut existing_future = iface.pending.take().unwrap_or_else(|| PacketFuture::new(config, &iface.handle));
129129
match Pin::new(&mut existing_future).poll(cx) {
130130
Poll::Pending => {
131-
println!("Pending");
131+
trace!("Pending");
132132
iface.pending = Some(existing_future);
133133
continue;
134134
//return Poll::Pending;
@@ -137,7 +137,7 @@ impl Stream for BridgeStream {
137137
return Poll::Ready(Some(Err(e)));
138138
}
139139
Poll::Ready(Ok(None)) => {
140-
println!("Interface has completed");
140+
trace!("Interface has completed");
141141
iface.complete = true;
142142
continue;
143143
}
@@ -151,21 +151,20 @@ impl Stream for BridgeStream {
151151
std::cmp::min(ts, *p.timestamp())
152152
}).or(Some(*p.timestamp()));
153153
}
154-
println!("Adding {} packets to current", v.len());
154+
trace!("Adding {} packets to current", v.len());
155155
iface.current.extend(v);
156156
}
157157
}
158158
}
159159

160-
println!("EOL {:?}", interfaces.len());
161160

162161
let res = gather_packets(interfaces, gather_to);
163162

164163
interfaces.retain(|iface| { //drop the complete interfaces
165164
return !iface.complete;
166165
});
167166

168-
if res.is_empty() {
167+
if res.is_empty() && interfaces.is_empty() {
169168
return Poll::Ready(None);
170169
} else {
171170
return Poll::Ready(Some(Ok(res)));

0 commit comments

Comments
 (0)