make skip a provided method of RingBufferRead

src/ringbuffer_trait.rs

@@ -71,7 +71,10 @@ pub trait RingBufferRead<T>: RingBuffer<T> + IntoIterator<Item = T> {
 
     /// dequeues the top item off the queue, but does not return it. Instead it is dropped.
     /// If the ringbuffer is empty, this function is a nop.
-    fn skip(&mut self);
+    #[inline]
+    fn skip(&mut self) {
+        let _ = self.dequeue();
+    }
 
     /// Returns an iterator over the elements in the ringbuffer,
     /// dequeueing elements as they are iterated over.
@@ -420,17 +423,6 @@ pub use iter::{
     RingBufferDrainingIterator, RingBufferIntoIterator, RingBufferIterator, RingBufferMutIterator,
 };
 
-/// Implement various functions on implementors of [`RingBufferRead`].
-/// This is to avoid duplicate code.
-macro_rules! impl_ringbuffer_read {
-    () => {
-        #[inline]
-        fn skip(&mut self) {
-            let _ = self.dequeue().map(drop);
-        }
-    };
-}
-
 /// Implement various functions on implementors of [`RingBuffer`].
 /// This is to avoid duplicate code.
 macro_rules! impl_ringbuffer {

src/with_alloc/alloc_ringbuffer.rs

@@ -270,8 +270,6 @@ impl<T, SIZE: RingbufferSize> RingBufferRead<T> for AllocRingBuffer<T, SIZE> {
             unsafe { Some(ptr::read(res)) }
         }
     }
-
-    impl_ringbuffer_read!();
 }
 
 impl<T, SIZE: RingbufferSize> Extend<T> for AllocRingBuffer<T, SIZE> {

src/with_alloc/vecdeque.rs

@@ -165,8 +165,6 @@ impl<T> RingBufferRead<T> for GrowableAllocRingBuffer<T> {
     fn dequeue(&mut self) -> Option<T> {
         self.pop_front()
     }
-
-    impl_ringbuffer_read!();
 }
 
 impl<T> RingBufferWrite<T> for GrowableAllocRingBuffer<T> {

src/with_const_generics.rs

@@ -238,8 +238,6 @@ impl<T, const CAP: usize> RingBufferRead<T> for ConstGenericRingBuffer<T, CAP> {
             unsafe { Some(res.assume_init()) }
         }
     }
-
-    impl_ringbuffer_read!();
 }
 
 impl<T, const CAP: usize> Extend<T> for ConstGenericRingBuffer<T, CAP> {