Skip to content

Commit a1a157f

Browse files
committed
refactor: change casts in GuestMemorySliceIter::do_next
Declare `len` as type `usize`, which reduce the number of casts between usize and u64 (the variable is most often uses as a usize, and only once needs to be u64). Signed-off-by: Patrick Roy <[email protected]>
1 parent c81d7e3 commit a1a157f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/guest_memory.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -517,15 +517,15 @@ impl<'a, M: GuestMemory + ?Sized> GuestMemorySliceIterator<'a, M> {
517517
};
518518

519519
let cap = region.len() - start.raw_value();
520-
let len = std::cmp::min(cap, self.count as GuestUsize);
520+
let len = std::cmp::min(cap as usize, self.count);
521521

522-
self.count -= len as usize;
522+
self.count -= len;
523523
self.addr = match self.addr.overflowing_add(len as GuestUsize) {
524524
(x @ GuestAddress(0), _) | (x, false) => x,
525525
(_, true) => return Some(Err(Error::GuestAddressOverflow)),
526526
};
527527

528-
Some(region.get_slice(start, len as usize))
528+
Some(region.get_slice(start, len))
529529
}
530530
}
531531

0 commit comments

Comments
 (0)