Skip to content

Commit 20f4727

Browse files
committed
remove more unneeded usages of std::io::Cursor
1 parent eee7428 commit 20f4727

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

capnp/src/serialize_packed.rs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,6 @@ pub fn write_message<W, A>(write: &mut W, message: &crate::message::Builder<A>)
362362
mod tests {
363363
use std::io::{Write, Read};
364364

365-
use std::io::Cursor;
366365
use quickcheck::{quickcheck, TestResult};
367366

368367
use crate::message::{ReaderOptions};
@@ -431,11 +430,10 @@ mod tests {
431430
fn round_trip(segments: Vec<Vec<crate::Word>>) -> TestResult {
432431
use crate::message::ReaderSegments;
433432
if segments.len() == 0 { return TestResult::discard(); }
434-
let mut cursor = Cursor::new(Vec::new());
433+
let mut buf: Vec<u8> = Vec::new();
435434

436-
write_message_segments(&mut PackedWrite { inner: &mut cursor }, &segments);
437-
cursor.set_position(0);
438-
let message = read_message(&mut cursor, ReaderOptions::new()).unwrap();
435+
write_message_segments(&mut PackedWrite { inner: &mut buf }, &segments);
436+
let message = read_message(&mut &buf[..], ReaderOptions::new()).unwrap();
439437
let result_segments = message.into_segments();
440438

441439
TestResult::from_bool(segments.iter().enumerate().all(|(i, segment)| {
@@ -512,10 +510,8 @@ mod tests {
512510
0, 0, 0, 0, 0, 0, 0, 0,
513511
0, 0, 0, 0, 0, 0, 0, 0]);
514512

515-
let mut cursor = Cursor::new(packed_buf);
516-
517513
// At one point, this failed due to serialize::read_message()
518514
// reading the segment table only one word at a time.
519-
read_message(&mut cursor, Default::default()).unwrap();
515+
read_message(&mut &packed_buf[..], Default::default()).unwrap();
520516
}
521517
}

0 commit comments

Comments
 (0)