renet
/
renet
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove RingBuffer::set_len(). 

set_len() with a positive length change is better represented
with enqueue(), and set_len() with a negative length change
is not really a valid operation on its own.
v0.7.x
whitequark 2017-09-07 23:53:32 +00:00
parent 1102bd94e7
commit 844843c4bc
1 changed files with 2 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/storage/ring_buffer.rs
View File

@ -64,17 +64,6 @@ impl<'a, T: 'a> RingBuffer<'a, T> {
self.length
}
/// Set the current number of elements in the ring buffer.
///
/// The newly added elements (if any) retain their old value.
///
/// # Panics
/// This function panics if the new length is greater than capacity.
pub fn set_len(&mut self, length: usize) {
assert!(length <= self.capacity());
self.length = length
}
/// Return the number of elements that can be added to the ring buffer.
pub fn window(&self) -> usize {
self.capacity() - self.len()
@ -297,14 +286,14 @@ mod test {
assert_eq!(ring.capacity(), 2);
assert_eq!(ring.window(), 2);
ring.set_len(1);
ring.length = 1;
assert!(!ring.empty());
assert!(!ring.full());
assert_eq!(ring.len(), 1);
assert_eq!(ring.capacity(), 2);
assert_eq!(ring.window(), 1);
ring.set_len(2);
ring.length = 2;
assert!(!ring.empty());
assert!(ring.full());
assert_eq!(ring.len(), 2);