Tidy up feature flag documentation

Author: rjsberry

Cargo.toml

@@ -53,7 +53,7 @@ no-dev-version = true
 tag-name = "{{version}}"
 
 [package.metadata.docs.rs]
-all-features = true
+all-features = ["arbitrary", "quickcheck", "serde", "rayon"]
 rustdoc-args = ["--cfg", "docsrs"]
 
 [workspace]

src/lib.rs

@@ -114,6 +114,7 @@ mod macros;
 mod equivalent;
 mod mutable_keys;
 #[cfg(feature = "serde")]
+#[cfg_attr(docsrs, doc(cfg(feature = "serde")))]
 mod serde;
 #[cfg(feature = "serde")]
 #[cfg_attr(docsrs, doc(cfg(feature = "serde")))]
@@ -126,6 +127,7 @@ pub mod set;
 // Placed after `map` and `set` so new `rayon` methods on the types
 // are documented after the "normal" methods.
 #[cfg(feature = "rayon")]
+#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 mod rayon;
 
 #[cfg(feature = "rustc-rayon")]

src/rayon/map.rs

@@ -3,8 +3,6 @@
 //! You will rarely need to interact with this module directly unless you need to name one of the
 //! iterator types.
 
-#![cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
-
 use super::collect;
 use rayon::iter::plumbing::{Consumer, ProducerCallback, UnindexedConsumer};
 use rayon::prelude::*;

src/rayon/set.rs

@@ -3,8 +3,6 @@
 //! You will rarely need to interact with this module directly unless you need to name one of the
 //! iterator types.
 
-#![cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
-
 use super::collect;
 use rayon::iter::plumbing::{Consumer, ProducerCallback, UnindexedConsumer};
 use rayon::prelude::*;
@@ -22,7 +20,6 @@ use crate::IndexSet;
 
 type Bucket<T> = crate::Bucket<T, ()>;
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<T, S> IntoParallelIterator for IndexSet<T, S>
 where
     T: Send,
@@ -37,7 +34,6 @@ where
     }
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<T> IntoParallelIterator for Box<Slice<T>>
 where
     T: Send,
@@ -80,7 +76,6 @@ impl<T: Send> IndexedParallelIterator for IntoParIter<T> {
     indexed_parallel_iterator_methods!(Bucket::key);
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<'a, T, S> IntoParallelIterator for &'a IndexSet<T, S>
 where
     T: Sync,
@@ -95,7 +90,6 @@ where
     }
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<'a, T> IntoParallelIterator for &'a Slice<T>
 where
     T: Sync,
@@ -144,7 +138,6 @@ impl<T: Sync> IndexedParallelIterator for ParIter<'_, T> {
     indexed_parallel_iterator_methods!(Bucket::key_ref);
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<'a, T, S> ParallelDrainRange<usize> for &'a mut IndexSet<T, S>
 where
     T: Send,
@@ -585,7 +578,6 @@ where
     }
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<T, S> FromParallelIterator<T> for IndexSet<T, S>
 where
     T: Eq + Hash + Send,
@@ -605,7 +597,6 @@ where
     }
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<T, S> ParallelExtend<T> for IndexSet<T, S>
 where
     T: Eq + Hash + Send,
@@ -621,7 +612,6 @@ where
     }
 }
 
-#[cfg_attr(docsrs, doc(cfg(feature = "rayon")))]
 impl<'a, T: 'a, S> ParallelExtend<&'a T> for IndexSet<T, S>
 where
     T: Copy + Eq + Hash + Send + Sync,

src/serde.rs

@@ -1,5 +1,3 @@
-#![cfg_attr(docsrs, doc(cfg(feature = "serde")))]
-
 use serde::de::value::{MapDeserializer, SeqDeserializer};
 use serde::de::{
     Deserialize, Deserializer, Error, IntoDeserializer, MapAccess, SeqAccess, Visitor,

src/serde_seq.rs

@@ -18,8 +18,6 @@
 //! }
 //! ```
 
-#![cfg_attr(docsrs, doc(cfg(feature = "serde")))]
-
 use serde::de::{Deserialize, Deserializer, SeqAccess, Visitor};
 use serde::ser::{Serialize, Serializer};