Chris Morgan  ›  Git  ›  anymap  ›  blobdiff
HashMap feature parity: impl Extend, notes on more
[anymap] / src / lib.rs
diff --git a/src/lib.rs b/src/lib.rs
index 1ad840b4b3f4c53d7012f7ea584c930e9c2f3ee3..d4f227a293da168d6b4ffa0119ffc1c8aa7946f5 100644 (file)
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -4,8 +4,19 @@
 
 #![warn(missing_docs, unused_results)]
 
 
 #![warn(missing_docs, unused_results)]
 
-use std::any::{Any, TypeId};
-use std::marker::PhantomData;
+#![cfg_attr(not(feature = "std"), no_std)]
+
+use core::any::{Any, TypeId};
+use core::marker::PhantomData;
+
+#[cfg(not(any(feature = "std", feature = "hashbrown")))]
+compile_error!("anymap: you must enable the 'std' feature or the 'hashbrown' feature");
+
+#[cfg(not(feature = "std"))]
+extern crate alloc;
+
+#[cfg(not(feature = "std"))]
+use alloc::boxed::Box;
 
 use raw::RawMap;
 use any::{UncheckedAnyExt, IntoBox};
 
 use raw::RawMap;
 use any::{UncheckedAnyExt, IntoBox};
@@ -60,6 +71,10 @@ macro_rules! impl_common_methods {
                 self.$field.shrink_to_fit()
             }
 
                 self.$field.shrink_to_fit()
             }
 
+            // Additional stable methods (as of 1.60.0-nightly) that could be added:
+            // try_reserve(&mut self, additional: usize) -> Result<(), TryReserveError>    (1.57.0)
+            // shrink_to(&mut self, min_capacity: usize)                                   (1.56.0)
+
             /// Returns the number of items in the collection.
             #[inline]
             pub fn len(&self) -> usize {
             /// Returns the number of items in the collection.
             #[inline]
             pub fn len(&self) -> usize {
@@ -95,7 +110,7 @@ pub mod raw;
 /// type-safe access to those values.
 ///
 /// The type parameter `A` allows you to use a different value type; normally you will want it to
 /// type-safe access to those values.
 ///
 /// The type parameter `A` allows you to use a different value type; normally you will want it to
-/// be `std::any::Any`, but there are other choices:
+/// be `core::any::Any` (also known as `std::any::Any`), but there are other choices:
 ///
 /// - If you want the entire map to be cloneable, use `CloneAny` instead of `Any`; with that, you
 ///   can only add types that implement `Clone` to the map.
 ///
 /// - If you want the entire map to be cloneable, use `CloneAny` instead of `Any`; with that, you
 ///   can only add types that implement `Clone` to the map.
@@ -104,9 +119,9 @@ pub mod raw;
 ///
 /// Cumulatively, there are thus six forms of map:
 ///
 ///
 /// Cumulatively, there are thus six forms of map:
 ///
-/// - <code>[Map]&lt;dyn [std::any::Any]&gt;</code>, also spelled [`AnyMap`] for convenience.
-/// - <code>[Map]&lt;dyn [std::any::Any] + Send&gt;</code>
-/// - <code>[Map]&lt;dyn [std::any::Any] + Send + Sync&gt;</code>
+/// - <code>[Map]&lt;dyn [core::any::Any]&gt;</code>, also spelled [`AnyMap`] for convenience.
+/// - <code>[Map]&lt;dyn [core::any::Any] + Send&gt;</code>
+/// - <code>[Map]&lt;dyn [core::any::Any] + Send + Sync&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny]&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny] + Send&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny] + Send + Sync&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny]&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny] + Send&gt;</code>
 /// - <code>[Map]&lt;dyn [CloneAny] + Send + Sync&gt;</code>
@@ -114,7 +129,7 @@ pub mod raw;
 /// ## Example
 ///
 /// (Here using the [`AnyMap`] convenience alias; the first line could use
 /// ## Example
 ///
 /// (Here using the [`AnyMap`] convenience alias; the first line could use
-/// <code>[anymap::Map][Map]::&lt;[std::any::Any]&gt;::new()</code> instead if desired.)
+/// <code>[anymap::Map][Map]::&lt;[core::any::Any]&gt;::new()</code> instead if desired.)
 ///
 /// ```rust
 /// let mut data = anymap::AnyMap::new();
 ///
 /// ```rust
 /// let mut data = anymap::AnyMap::new();
@@ -192,6 +207,8 @@ impl<A: ?Sized + UncheckedAnyExt> Map<A> {
         }
     }
 
         }
     }
 
+    // rustc 1.60.0-nightly has another method try_insert that would be nice to add when stable.
+
     /// Removes the `T` value from the collection,
     /// returning it if there was one or `None` if there was not.
     #[inline]
     /// Removes the `T` value from the collection,
     /// returning it if there was one or `None` if there was not.
     #[inline]
@@ -222,6 +239,15 @@ impl<A: ?Sized + UncheckedAnyExt> Map<A> {
     }
 }
 
     }
 }
 
+impl<A: ?Sized + UncheckedAnyExt> Extend<Box<A>> for Map<A> {
+    #[inline]
+    fn extend<T: IntoIterator<Item = Box<A>>>(&mut self, iter: T) {
+        for item in iter {
+            let _ = unsafe { self.raw.insert(item.type_id(), item) };
+        }
+    }
+}
+
 impl<A: ?Sized + UncheckedAnyExt> AsRef<RawMap<A>> for Map<A> {
     #[inline]
     fn as_ref(&self) -> &RawMap<A> {
 impl<A: ?Sized + UncheckedAnyExt> AsRef<RawMap<A>> for Map<A> {
     #[inline]
     fn as_ref(&self) -> &RawMap<A> {
@@ -444,7 +470,7 @@ mod tests {
         fn assert_send<T: Send>() { }
         fn assert_sync<T: Sync>() { }
         fn assert_clone<T: Clone>() { }
         fn assert_send<T: Send>() { }
         fn assert_sync<T: Sync>() { }
         fn assert_clone<T: Clone>() { }
-        fn assert_debug<T: ::std::fmt::Debug>() { }
+        fn assert_debug<T: ::core::fmt::Debug>() { }
         assert_send::<Map<dyn Any + Send>>();
         assert_send::<Map<dyn Any + Send + Sync>>();
         assert_sync::<Map<dyn Any + Send + Sync>>();
         assert_send::<Map<dyn Any + Send>>();
         assert_send::<Map<dyn Any + Send + Sync>>();
         assert_sync::<Map<dyn Any + Send + Sync>>();