Merge pull request #587 from sgued/fix-docs

Fix broken tests and update CI to test more features

Author: sgued

.github/workflows/build.yml

@@ -48,7 +48,7 @@ jobs:
           components: miri
 
       - name: Run miri
-        run: MIRIFLAGS=-Zmiri-ignore-leaks cargo miri test
+        run: MIRIFLAGS=-Zmiri-ignore-leaks cargo miri test --features="alloc,defmt,mpmc_large,portable-atomic-critical-section,serde,ufmt,bytes"
 
   # Run cargo test
   test:
@@ -84,7 +84,7 @@ jobs:
           toolchain: stable
 
       - name: Run cargo test
-        run: cargo test
+        run: cargo test --features="alloc,defmt,mpmc_large,portable-atomic-critical-section,serde,ufmt,bytes"
 
   # Run cargo fmt --check
   style:

src/bytes.rs

@@ -61,21 +61,21 @@ mod tests {
     #[test]
     #[should_panic]
     fn buf_mut_advance_mut_out_of_bounds_view() {
-        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8>::new();
+        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8, u8>::new();
         unsafe { vec.advance_mut(9) };
     }
 
     #[test]
     fn buf_mut_remaining_mut_view() {
-        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8>::new();
+        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8, u8>::new();
         assert_eq!(vec.remaining_mut(), 8);
         vec.push(42).unwrap();
         assert_eq!(vec.remaining_mut(), 7);
     }
 
     #[test]
     fn buf_mut_chunk_mut_view() {
-        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8>::new();
+        let vec: &mut VecView<u8, u8> = &mut Vec::<u8, 8, u8>::new();
         assert_eq!(vec.chunk_mut().len(), 8);
         unsafe { vec.advance_mut(1) };
         assert_eq!(vec.chunk_mut().len(), 7);

src/pool/arc.rs

@@ -3,6 +3,7 @@
 //! # Example usage
 //!
 //! ```
+//! use core::ptr::addr_of_mut;
 //! use heapless::{arc_pool, pool::arc::{Arc, ArcBlock}};
 //!
 //! arc_pool!(MyArcPool: u128);
@@ -45,6 +46,7 @@
 //! to the `ArcPool`. This requires an intermediate `const` value as shown below:
 //!
 //! ```
+//! use core::ptr::addr_of_mut;
 //! use heapless::{arc_pool, pool::arc::ArcBlock};
 //!
 //! arc_pool!(MyArcPool: u128);
@@ -54,7 +56,7 @@
 //! let blocks: &'static mut [ArcBlock<u128>] = {
 //!     const BLOCK: ArcBlock<u128> = ArcBlock::new(); // <=
 //!     static mut BLOCKS: [ArcBlock<u128>; POOL_CAPACITY] = [BLOCK; POOL_CAPACITY];
-//!     unsafe { addr_of_mut!(BLOCK).as_mut().unwrap()S }
+//!     unsafe { addr_of_mut!(BLOCKS).as_mut().unwrap() }
 //! };
 //!
 //! for block in blocks {

src/pool/boxed.rs

@@ -61,6 +61,7 @@
 //! to the `BoxPool`. This requires an intermediate `const` value as shown below:
 //!
 //! ```
+//! use core::ptr::addr_of_mut;
 //! use heapless::{box_pool, pool::boxed::BoxBlock};
 //!
 //! box_pool!(MyBoxPool: u128);

src/pool/object.rs

@@ -3,6 +3,7 @@
 //! # Example usage
 //!
 //! ```
+//! use core::ptr::addr_of_mut;
 //! use heapless::{object_pool, pool::object::{Object, ObjectBlock}};
 //!
 //! object_pool!(MyObjectPool: [u8; 128]);
@@ -46,6 +47,7 @@
 //! to the `ObjectPool`. This requires an intermediate `const` value as shown below:
 //!
 //! ```
+//! use core::ptr::addr_of_mut;
 //! use heapless::{object_pool, pool::object::ObjectBlock};
 //!
 //! object_pool!(MyObjectPool: [u8; 128]);
@@ -55,7 +57,7 @@
 //! let blocks: &'static mut [ObjectBlock<[u8; 128]>] = {
 //!     const BLOCK: ObjectBlock<[u8; 128]> = ObjectBlock::new([0; 128]); // <=
 //!     static mut BLOCKS: [ObjectBlock<[u8; 128]>; POOL_CAPACITY] = [BLOCK; POOL_CAPACITY];
-//!     unsafe { addr_of_mut!(BLOCK).as_mut().unwrap()S }
+//!     unsafe { addr_of_mut!(BLOCKS).as_mut().unwrap() }
 //! };
 //!
 //! for block in blocks {