Merge pull request #455 from reitermarkus/fix-ci

Fix CI.

Author: Dirbaio

Cargo.toml

@@ -52,6 +52,7 @@ stable_deref_trait = { version = "1", default-features = false }
 
 [dev-dependencies]
 ufmt = "0.2"
+static_assertions = "1.1.0"
 
 [package.metadata.docs.rs]
 features = ["ufmt", "serde", "defmt-03", "mpmc_large", "portable-atomic-critical-section"]

cfail/ui/not-send.rs

@@ -564,9 +564,13 @@ where
 
 #[cfg(test)]
 mod tests {
-    use std::vec::Vec;
+    use static_assertions::assert_not_impl_any;
 
-    use crate::binary_heap::{BinaryHeap, Max, Min};
+    use super::{BinaryHeap, Max, Min};
+
+    // Ensure a `BinaryHeap` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(BinaryHeap<*const (), Max, 4>: Send);
+    assert_not_impl_any!(BinaryHeap<*const (), Min, 4>: Send);
 
     #[test]
     fn static_new() {

cfail/ui/not-send.stderr

@@ -745,7 +745,12 @@ where
 
 #[cfg(test)]
 mod tests {
-    use crate::Deque;
+    use static_assertions::assert_not_impl_any;
+
+    use super::Deque;
+
+    // Ensure a `Deque` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(Deque<*const (), 4>: Send);
 
     #[test]
     fn static_new() {

src/binary_heap.rs

@@ -433,10 +433,16 @@ impl<'a, T, const N: usize> Iterator for OldestOrdered<'a, T, N> {
 
 #[cfg(test)]
 mod tests {
-    use crate::HistoryBuffer;
     use core::fmt::Debug;
     use core::sync::atomic::{AtomicUsize, Ordering};
 
+    use static_assertions::assert_not_impl_any;
+
+    use super::HistoryBuffer;
+
+    // Ensure a `HistoryBuffer` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(HistoryBuffer<*const (), 4>: Send);
+
     #[test]
     fn new() {
         let x: HistoryBuffer<u8, 4> = HistoryBuffer::new_with(1);

src/deque.rs

@@ -2,7 +2,6 @@ use core::{
     borrow::Borrow,
     fmt,
     hash::{BuildHasher, Hash},
-    iter::FromIterator,
     mem,
     num::NonZeroU32,
     ops, slice,
@@ -1263,10 +1262,17 @@ where
 
 #[cfg(test)]
 mod tests {
-    use crate::{indexmap::Entry, FnvIndexMap};
-
     use core::mem;
 
+    use static_assertions::assert_not_impl_any;
+
+    use super::{BuildHasherDefault, Entry, FnvIndexMap, IndexMap};
+
+    // Ensure a `IndexMap` containing `!Send` keys stays `!Send` itself.
+    assert_not_impl_any!(IndexMap<*const (), (), BuildHasherDefault<()>, 4>: Send);
+    // Ensure a `IndexMap` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(IndexMap<(), *const (), BuildHasherDefault<()>, 4>: Send);
+
     #[test]
     fn size() {
         const CAP: usize = 4;

src/histbuf.rs

@@ -1,12 +1,13 @@
-use crate::indexmap::{self, IndexMap};
 use core::{
     borrow::Borrow,
     fmt,
     hash::{BuildHasher, Hash},
-    iter::FromIterator,
 };
+
 use hash32::{BuildHasherDefault, FnvHasher};
 
+use crate::indexmap::{self, IndexMap};
+
 /// An [`IndexSet`] using the default FNV hasher.
 ///
 /// A list of all Methods and Traits available for `FnvIndexSet` can be found in
@@ -660,3 +661,13 @@ where
         }
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use static_assertions::assert_not_impl_any;
+
+    use super::{BuildHasherDefault, IndexSet};
+
+    // Ensure a `IndexSet` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(IndexSet<*const (), BuildHasherDefault<()>, 4>: Send);
+}

src/indexmap.rs

@@ -1,5 +1,6 @@
+use core::{borrow::Borrow, fmt, mem, ops, slice};
+
 use crate::Vec;
-use core::{borrow::Borrow, fmt, iter::FromIterator, mem, ops, slice};
 
 /// A fixed capacity map/dictionary that performs lookups via linear search.
 ///
@@ -492,7 +493,14 @@ where
 
 #[cfg(test)]
 mod test {
-    use crate::LinearMap;
+    use static_assertions::assert_not_impl_any;
+
+    use super::LinearMap;
+
+    // Ensure a `LinearMap` containing `!Send` keys stays `!Send` itself.
+    assert_not_impl_any!(LinearMap<*const (), (), 4>: Send);
+    // Ensure a `LinearMap` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(LinearMap<(), *const (), 4>: Send);
 
     #[test]
     fn static_new() {

src/indexset.rs

@@ -294,7 +294,12 @@ unsafe fn enqueue<T>(
 
 #[cfg(test)]
 mod tests {
-    use super::Q2;
+    use static_assertions::assert_not_impl_any;
+
+    use super::{MpMcQueue, Q2};
+
+    // Ensure a `MpMcQueue` containing `!Send` values stays `!Send` itself.
+    assert_not_impl_any!(MpMcQueue<*const (), 4>: Send);
 
     #[test]
     fn sanity() {