Resolve the std/hashbrown conflict situation

[anymap] / src / lib.rs
diff --git a/src/lib.rs b/src/lib.rs

index 15a21bce7f9100dd9253213af9408233af88fc48..9c57f09e34c05273c770c74cf23aa234254206af 100644 (file)
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1,44 +1,152 @@
-//! This crate provides the `AnyMap` type, a safe and convenient store for one value of each type.
+//! This crate provides a safe and convenient store for one value of each type.
+//!
+//! Your starting point is [`Map`]. It has an example.
+//!
+//! # Cargo features
+//!
+//! This crate has two independent features, each of which provides an implementation providing
+//! types `Map`, `AnyMap`, `OccupiedEntry`, `VacantEntry`, `Entry` and `RawMap`:
+//!
+#![cfg_attr(feature = "std", doc = " - **std** (default, *enabled* in this build):")]
+#![cfg_attr(not(feature = "std"), doc = " - **std** (default, *disabled* in this build):")]
+//!   an implementation using `std::collections::hash_map`, placed in the crate root
+//!   (e.g. `anymap::AnyMap`).
+//!
+#![cfg_attr(feature = "hashbrown", doc = " - **hashbrown** (optional; *enabled* in this build):")]
+#![cfg_attr(not(feature = "hashbrown"), doc = " - **hashbrown** (optional; *disabled* in this build):")]
+//!   an implementation using `alloc` and `hashbrown::hash_map`, placed in a module `hashbrown`
+//!   (e.g. `anymap::hashbrown::AnyMap`).
  
  
-#![cfg_attr(all(feature = "bench", test), feature(test))]
  #![warn(missing_docs, unused_results)]
  
  #![warn(missing_docs, unused_results)]
  
-#[cfg(all(feature = "bench", test))]
-extern crate test;
+#![cfg_attr(not(feature = "std"), no_std)]
+
+use core::convert::TryInto;
+use core::hash::Hasher;
+
+#[cfg(not(feature = "std"))]
+extern crate alloc;
+
+pub use crate::any::CloneAny;
+
+mod any;
+
+#[cfg(any(feature = "std", feature = "hashbrown"))]
+macro_rules! everything {
+    ($example_init:literal, $($parent:ident)::+ $(, $entry_generics:ty)?) => {
+        use core::any::{Any, TypeId};
+        use core::hash::BuildHasherDefault;
+        use core::marker::PhantomData;
+
+        #[cfg(not(feature = "std"))]
+        use alloc::boxed::Box;
+
+        use ::$($parent)::+::hash_map::{self, HashMap};
+
+        use crate::any::{Downcast, IntoBox};
+
+        /// Raw access to the underlying `HashMap`.
+        ///
+        /// This alias is provided for convenience because of the ugly third generic parameter.
+        pub type RawMap<A> = HashMap<TypeId, Box<A>, BuildHasherDefault<TypeIdHasher>>;
+
+        /// A collection containing zero or one values for any given type and allowing convenient,
+        /// 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 `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.
+        /// - You can add on `+ Send` or `+ Send + Sync` (e.g. `Map<dyn Any + Send>`) to add those
+        ///   auto traits.
+        ///
+        /// Cumulatively, there are thus six forms of map:
+        ///
+        /// - <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>
+        ///
+        /// ## Example
+        ///
+        /// (Here using the [`AnyMap`] convenience alias; the first line could use
+        /// <code>[anymap::Map][Map]::&lt;[core::any::Any]&gt;::new()</code> instead if desired.)
+        ///
+        /// ```rust
+        #[doc = $example_init]
+        /// assert_eq!(data.get(), None::<&i32>);
+        /// data.insert(42i32);
+        /// assert_eq!(data.get(), Some(&42i32));
+        /// data.remove::<i32>();
+        /// assert_eq!(data.get::<i32>(), None);
+        ///
+        /// #[derive(Clone, PartialEq, Debug)]
+        /// struct Foo {
+        ///     str: String,
+        /// }
+        ///
+        /// assert_eq!(data.get::<Foo>(), None);
+        /// data.insert(Foo { str: format!("foo") });
+        /// assert_eq!(data.get(), Some(&Foo { str: format!("foo") }));
+        /// data.get_mut::<Foo>().map(|foo| foo.str.push('t'));
+        /// assert_eq!(&*data.get::<Foo>().unwrap().str, "foot");
+        /// ```
+        ///
+        /// Values containing non-static references are not permitted.
+        #[derive(Debug)]
+        pub struct Map<A: ?Sized + Downcast = dyn Any> {
+            raw: RawMap<A>,
+        }
  
  
-use std::any::TypeId;
-use std::marker::PhantomData;
+        // #[derive(Clone)] would want A to implement Clone, but in reality only Box<A> can.
+        impl<A: ?Sized + Downcast> Clone for Map<A> where Box<A>: Clone {
+            #[inline]
+            fn clone(&self) -> Map<A> {
+                Map {
+                    raw: self.raw.clone(),
+                }
+            }
+        }
  
  
-use raw::RawMap;
-use any::{UncheckedAnyExt, IntoBox, Any};
+        /// The most common type of `Map`: just using `Any`; <code>[Map]&lt;dyn [Any]&gt;</code>.
+        ///
+        /// Why is this a separate type alias rather than a default value for `Map<A>`?
+        /// `Map::new()` doesn’t seem to be happy to infer that it should go with the default
+        /// value. It’s a bit sad, really. Ah well, I guess this approach will do.
+        pub type AnyMap = Map<dyn Any>;
  
  
-macro_rules! impl_common_methods {
-    (
-        field: $t:ident.$field:ident;
-        new() => $new:expr;
-        with_capacity($with_capacity_arg:ident) => $with_capacity:expr;
-    ) => {
-        impl<A: ?Sized + UncheckedAnyExt> $t<A> {
+        impl<A: ?Sized + Downcast> Default for Map<A> {
+            #[inline]
+            fn default() -> Map<A> {
+                Map::new()
+            }
+        }
+
+        impl<A: ?Sized + Downcast> Map<A> {
              /// Create an empty collection.
              #[inline]
              /// Create an empty collection.
              #[inline]
-            pub fn new() -> $t<A> {
-                $t {
-                    $field: $new,
+            pub fn new() -> Map<A> {
+                Map {
+                    raw: RawMap::with_hasher(Default::default()),
                  }
              }
  
              /// Creates an empty collection with the given initial capacity.
              #[inline]
                  }
              }
  
              /// Creates an empty collection with the given initial capacity.
              #[inline]
-            pub fn with_capacity($with_capacity_arg: usize) -> $t<A> {
-                $t {
-                    $field: $with_capacity,
+            pub fn with_capacity(capacity: usize) -> Map<A> {
+                Map {
+                    raw: RawMap::with_capacity_and_hasher(capacity, Default::default()),
                  }
              }
  
              /// Returns the number of elements the collection can hold without reallocating.
              #[inline]
              pub fn capacity(&self) -> usize {
                  }
              }
  
              /// Returns the number of elements the collection can hold without reallocating.
              #[inline]
              pub fn capacity(&self) -> usize {
-                self.$field.capacity()
+                self.raw.capacity()
              }
  
              /// Reserves capacity for at least `additional` more elements to be inserted
              }
  
              /// Reserves capacity for at least `additional` more elements to be inserted
@@ -50,7 +158,7 @@ macro_rules! impl_common_methods {
              /// Panics if the new allocation size overflows `usize`.
              #[inline]
              pub fn reserve(&mut self, additional: usize) {
              /// Panics if the new allocation size overflows `usize`.
              #[inline]
              pub fn reserve(&mut self, additional: usize) {
-                self.$field.reserve(additional)
+                self.raw.reserve(additional)
              }
  
              /// Shrinks the capacity of the collection as much as possible. It will drop
              }
  
              /// Shrinks the capacity of the collection as much as possible. It will drop
@@ -58,468 +166,481 @@ macro_rules! impl_common_methods {
              /// and possibly leaving some space in accordance with the resize policy.
              #[inline]
              pub fn shrink_to_fit(&mut self) {
              /// and possibly leaving some space in accordance with the resize policy.
              #[inline]
              pub fn shrink_to_fit(&mut self) {
-                self.$field.shrink_to_fit()
+                self.raw.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 {
-                self.$field.len()
+                self.raw.len()
              }
  
              /// Returns true if there are no items in the collection.
              #[inline]
              pub fn is_empty(&self) -> bool {
              }
  
              /// Returns true if there are no items in the collection.
              #[inline]
              pub fn is_empty(&self) -> bool {
-                self.$field.is_empty()
+                self.raw.is_empty()
              }
  
              /// Removes all items from the collection. Keeps the allocated memory for reuse.
              #[inline]
              pub fn clear(&mut self) {
              }
  
              /// Removes all items from the collection. Keeps the allocated memory for reuse.
              #[inline]
              pub fn clear(&mut self) {
-                self.$field.clear()
+                self.raw.clear()
              }
              }
-        }
-    }
-}
-
-pub mod any;
-pub mod raw;
  
  
-/// A collection containing zero or one values for any given type and allowing convenient,
-/// 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 `anymap::any::Any`, but there are other choices:
-///
-/// - If you want the entire map to be cloneable, use `CloneAny` instead of `Any`.
-/// - You can add on `+ Send` and/or `+ Sync` (e.g. `Map<Any + Send>`) to add those bounds.
-///
-/// ```rust
-/// # use anymap::AnyMap;
-/// let mut data = AnyMap::new();
-/// assert_eq!(data.get(), None::<&i32>);
-/// data.insert(42i32);
-/// assert_eq!(data.get(), Some(&42i32));
-/// data.remove::<i32>();
-/// assert_eq!(data.get::<i32>(), None);
-///
-/// #[derive(Clone, PartialEq, Debug)]
-/// struct Foo {
-///     str: String,
-/// }
-///
-/// assert_eq!(data.get::<Foo>(), None);
-/// data.insert(Foo { str: format!("foo") });
-/// assert_eq!(data.get(), Some(&Foo { str: format!("foo") }));
-/// data.get_mut::<Foo>().map(|foo| foo.str.push('t'));
-/// assert_eq!(&*data.get::<Foo>().unwrap().str, "foot");
-/// ```
-///
-/// Values containing non-static references are not permitted.
-#[derive(Debug)]
-pub struct Map<A: ?Sized + UncheckedAnyExt = Any> {
-    raw: RawMap<A>,
-}
-
-// #[derive(Clone)] would want A to implement Clone, but in reality it’s only Box<A> that can.
-impl<A: ?Sized + UncheckedAnyExt> Clone for Map<A> where Box<A>: Clone {
-    fn clone(&self) -> Map<A> {
-        Map {
-            raw: self.raw.clone(),
-        }
-    }
-}
-
-/// The most common type of `Map`: just using `Any`.
-///
-/// Why is this a separate type alias rather than a default value for `Map<A>`? `Map::new()`
-/// doesn’t seem to be happy to infer that it should go with the default value.
-/// It’s a bit sad, really. Ah well, I guess this approach will do.
-pub type AnyMap = Map<Any>;
-
-impl_common_methods! {
-    field: Map.raw;
-    new() => RawMap::new();
-    with_capacity(capacity) => RawMap::with_capacity(capacity);
-}
+            /// Returns a reference to the value stored in the collection for the type `T`,
+            /// if it exists.
+            #[inline]
+            pub fn get<T: IntoBox<A>>(&self) -> Option<&T> {
+                self.raw.get(&TypeId::of::<T>())
+                    .map(|any| unsafe { any.downcast_ref_unchecked::<T>() })
+            }
  
  
-impl<A: ?Sized + UncheckedAnyExt> Map<A> {
-    /// Returns a reference to the value stored in the collection for the type `T`, if it exists.
-    pub fn get<T: IntoBox<A>>(&self) -> Option<&T> {
-        self.raw.get(&TypeId::of::<T>())
-            .map(|any| unsafe { any.downcast_ref_unchecked::<T>() })
-    }
+            /// Returns a mutable reference to the value stored in the collection for the type `T`,
+            /// if it exists.
+            #[inline]
+            pub fn get_mut<T: IntoBox<A>>(&mut self) -> Option<&mut T> {
+                self.raw.get_mut(&TypeId::of::<T>())
+                    .map(|any| unsafe { any.downcast_mut_unchecked::<T>() })
+            }
  
  
-    /// Returns a mutable reference to the value stored in the collection for the type `T`,
-    /// if it exists.
-    pub fn get_mut<T: IntoBox<A>>(&mut self) -> Option<&mut T> {
-        self.raw.get_mut(&TypeId::of::<T>())
-            .map(|any| unsafe { any.downcast_mut_unchecked::<T>() })
-    }
+            /// Sets the value stored in the collection for the type `T`.
+            /// If the collection already had a value of type `T`, that value is returned.
+            /// Otherwise, `None` is returned.
+            #[inline]
+            pub fn insert<T: IntoBox<A>>(&mut self, value: T) -> Option<T> {
+                self.raw.insert(TypeId::of::<T>(), value.into_box())
+                    .map(|any| unsafe { *any.downcast_unchecked::<T>() })
+            }
  
  
-    /// Sets the value stored in the collection for the type `T`.
-    /// If the collection already had a value of type `T`, that value is returned.
-    /// Otherwise, `None` is returned.
-    pub fn insert<T: IntoBox<A>>(&mut self, value: T) -> Option<T> {
-        unsafe {
-            self.raw.insert(TypeId::of::<T>(), value.into_box())
-                .map(|any| *any.downcast_unchecked::<T>())
-        }
-    }
+            // rustc 1.60.0-nightly has another method try_insert that would be nice when stable.
  
  
-    /// Removes the `T` value from the collection,
-    /// returning it if there was one or `None` if there was not.
-    pub fn remove<T: IntoBox<A>>(&mut self) -> Option<T> {
-        self.raw.remove(&TypeId::of::<T>())
-            .map(|any| *unsafe { any.downcast_unchecked::<T>() })
-    }
+            /// Removes the `T` value from the collection,
+            /// returning it if there was one or `None` if there was not.
+            #[inline]
+            pub fn remove<T: IntoBox<A>>(&mut self) -> Option<T> {
+                self.raw.remove(&TypeId::of::<T>())
+                    .map(|any| *unsafe { any.downcast_unchecked::<T>() })
+            }
  
  
-    /// Returns true if the collection contains a value of type `T`.
-    #[inline]
-    pub fn contains<T: IntoBox<A>>(&self) -> bool {
-        self.raw.contains_key(&TypeId::of::<T>())
-    }
+            /// Returns true if the collection contains a value of type `T`.
+            #[inline]
+            pub fn contains<T: IntoBox<A>>(&self) -> bool {
+                self.raw.contains_key(&TypeId::of::<T>())
+            }
  
  
-    /// Gets the entry for the given type in the collection for in-place manipulation
-    pub fn entry<T: IntoBox<A>>(&mut self) -> Entry<A, T> {
-        match self.raw.entry(TypeId::of::<T>()) {
-            raw::Entry::Occupied(e) => Entry::Occupied(OccupiedEntry {
-                inner: e,
-                type_: PhantomData,
-            }),
-            raw::Entry::Vacant(e) => Entry::Vacant(VacantEntry {
-                inner: e,
-                type_: PhantomData,
-            }),
-        }
-    }
-}
+            /// Gets the entry for the given type in the collection for in-place manipulation
+            #[inline]
+            pub fn entry<T: IntoBox<A>>(&mut self) -> Entry<A, T> {
+                match self.raw.entry(TypeId::of::<T>()) {
+                    hash_map::Entry::Occupied(e) => Entry::Occupied(OccupiedEntry {
+                        inner: e,
+                        type_: PhantomData,
+                    }),
+                    hash_map::Entry::Vacant(e) => Entry::Vacant(VacantEntry {
+                        inner: e,
+                        type_: PhantomData,
+                    }),
+                }
+            }
  
  
-impl<A: ?Sized + UncheckedAnyExt> AsRef<RawMap<A>> for Map<A> {
-    fn as_ref(&self) -> &RawMap<A> {
-        &self.raw
-    }
-}
+            /// Get access to the raw hash map that backs this.
+            ///
+            /// This will seldom be useful, but it’s conceivable that you could wish to iterate
+            /// over all the items in the collection, and this lets you do that.
+            #[inline]
+            pub fn as_raw(&self) -> &RawMap<A> {
+                &self.raw
+            }
  
  
-impl<A: ?Sized + UncheckedAnyExt> AsMut<RawMap<A>> for Map<A> {
-    fn as_mut(&mut self) -> &mut RawMap<A> {
-        &mut self.raw
-    }
-}
+            /// Get mutable access to the raw hash map that backs this.
+            ///
+            /// This will seldom be useful, but it’s conceivable that you could wish to iterate
+            /// over all the items in the collection mutably, or drain or something, or *possibly*
+            /// even batch insert, and this lets you do that.
+            ///
+            /// # Safety
+            ///
+            /// If you insert any values to the raw map, the key (a `TypeId`) must match the
+            /// value’s type, or *undefined behaviour* will occur when you access those values.
+            ///
+            /// (*Removing* entries is perfectly safe.)
+            #[inline]
+            pub unsafe fn as_raw_mut(&mut self) -> &mut RawMap<A> {
+                &mut self.raw
+            }
  
  
-impl<A: ?Sized + UncheckedAnyExt> Into<RawMap<A>> for Map<A> {
-    fn into(self) -> RawMap<A> {
-        self.raw
-    }
-}
+            /// Convert this into the raw hash map that backs this.
+            ///
+            /// This will seldom be useful, but it’s conceivable that you could wish to consume all
+            /// the items in the collection and do *something* with some or all of them, and this
+            /// lets you do that, without the `unsafe` that `.as_raw_mut().drain()` would require.
+            #[inline]
+            pub fn into_raw(self) -> RawMap<A> {
+                self.raw
+            }
  
  
-/// A view into a single occupied location in an `Map`.
-pub struct OccupiedEntry<'a, A: ?Sized + UncheckedAnyExt, V: 'a> {
-    inner: raw::OccupiedEntry<'a, A>,
-    type_: PhantomData<V>,
-}
+            /// Construct a map from a collection of raw values.
+            ///
+            /// You know what? I can’t immediately think of any legitimate use for this, especially
+            /// because of the requirement of the `BuildHasherDefault<TypeIdHasher>` generic in the
+            /// map.
+            ///
+            /// Perhaps this will be most practical as `unsafe { Map::from_raw(iter.collect()) }`,
+            /// `iter` being an iterator over `(TypeId, Box<A>)` pairs. Eh, this method provides
+            /// symmetry with `into_raw`, so I don’t care if literally no one ever uses it. I’m not
+            /// even going to write a test for it, it’s so trivial.
+            ///
+            /// # Safety
+            ///
+            /// For all entries in the raw map, the key (a `TypeId`) must match the value’s type,
+            /// or *undefined behaviour* will occur when you access that entry.
+            #[inline]
+            pub unsafe fn from_raw(raw: RawMap<A>) -> Map<A> {
+                Self { raw }
+            }
+        }
  
  
-/// A view into a single empty location in an `Map`.
-pub struct VacantEntry<'a, A: ?Sized + UncheckedAnyExt, V: 'a> {
-    inner: raw::VacantEntry<'a, A>,
-    type_: PhantomData<V>,
-}
+        impl<A: ?Sized + Downcast> Extend<Box<A>> for Map<A> {
+            #[inline]
+            fn extend<T: IntoIterator<Item = Box<A>>>(&mut self, iter: T) {
+                for item in iter {
+                    let _ = self.raw.insert(Downcast::type_id(&*item), item);
+                }
+            }
+        }
  
  
-/// A view into a single location in an `Map`, which may be vacant or occupied.
-pub enum Entry<'a, A: ?Sized + UncheckedAnyExt, V: 'a> {
-    /// An occupied Entry
-    Occupied(OccupiedEntry<'a, A, V>),
-    /// A vacant Entry
-    Vacant(VacantEntry<'a, A, V>),
-}
+        /// A view into a single occupied location in an `Map`.
+        pub struct OccupiedEntry<'a, A: ?Sized + Downcast, V: 'a> {
+            inner: hash_map::OccupiedEntry<'a, TypeId, Box<A>, $($entry_generics)?>,
+            type_: PhantomData<V>,
+        }
  
  
-impl<'a, A: ?Sized + UncheckedAnyExt, V: IntoBox<A>> Entry<'a, A, V> {
-    /// Ensures a value is in the entry by inserting the default if empty, and returns
-    /// a mutable reference to the value in the entry.
-    pub fn or_insert(self, default: V) -> &'a mut V {
-        match self {
-            Entry::Occupied(inner) => inner.into_mut(),
-            Entry::Vacant(inner) => inner.insert(default),
+        /// A view into a single empty location in an `Map`.
+        pub struct VacantEntry<'a, A: ?Sized + Downcast, V: 'a> {
+            inner: hash_map::VacantEntry<'a, TypeId, Box<A>, $($entry_generics)?>,
+            type_: PhantomData<V>,
          }
          }
-    }
  
  
-    /// Ensures a value is in the entry by inserting the result of the default function if empty,
-    /// and returns a mutable reference to the value in the entry.
-    pub fn or_insert_with<F: FnOnce() -> V>(self, default: F) -> &'a mut V {
-        match self {
-            Entry::Occupied(inner) => inner.into_mut(),
-            Entry::Vacant(inner) => inner.insert(default()),
+        /// A view into a single location in an `Map`, which may be vacant or occupied.
+        pub enum Entry<'a, A: ?Sized + Downcast, V: 'a> {
+            /// An occupied Entry
+            Occupied(OccupiedEntry<'a, A, V>),
+            /// A vacant Entry
+            Vacant(VacantEntry<'a, A, V>),
          }
          }
-    }
-}
  
  
-impl<'a, A: ?Sized + UncheckedAnyExt, V: IntoBox<A>> OccupiedEntry<'a, A, V> {
-    /// Gets a reference to the value in the entry
-    pub fn get(&self) -> &V {
-        unsafe { self.inner.get().downcast_ref_unchecked() }
-    }
+        impl<'a, A: ?Sized + Downcast, V: IntoBox<A>> Entry<'a, A, V> {
+            /// Ensures a value is in the entry by inserting the default if empty, and returns
+            /// a mutable reference to the value in the entry.
+            #[inline]
+            pub fn or_insert(self, default: V) -> &'a mut V {
+                match self {
+                    Entry::Occupied(inner) => inner.into_mut(),
+                    Entry::Vacant(inner) => inner.insert(default),
+                }
+            }
  
  
-    /// Gets a mutable reference to the value in the entry
-    pub fn get_mut(&mut self) -> &mut V {
-        unsafe { self.inner.get_mut().downcast_mut_unchecked() }
-    }
+            /// Ensures a value is in the entry by inserting the result of the default function if
+            /// empty, and returns a mutable reference to the value in the entry.
+            #[inline]
+            pub fn or_insert_with<F: FnOnce() -> V>(self, default: F) -> &'a mut V {
+                match self {
+                    Entry::Occupied(inner) => inner.into_mut(),
+                    Entry::Vacant(inner) => inner.insert(default()),
+                }
+            }
  
  
-    /// Converts the OccupiedEntry into a mutable reference to the value in the entry
-    /// with a lifetime bound to the collection itself
-    pub fn into_mut(self) -> &'a mut V {
-        unsafe { self.inner.into_mut().downcast_mut_unchecked() }
-    }
+            /// Ensures a value is in the entry by inserting the default value if empty,
+            /// and returns a mutable reference to the value in the entry.
+            #[inline]
+            pub fn or_default(self) -> &'a mut V where V: Default {
+                match self {
+                    Entry::Occupied(inner) => inner.into_mut(),
+                    Entry::Vacant(inner) => inner.insert(Default::default()),
+                }
+            }
  
  
-    /// Sets the value of the entry, and returns the entry's old value
-    pub fn insert(&mut self, value: V) -> V {
-        unsafe { *self.inner.insert(value.into_box()).downcast_unchecked() }
-    }
+            /// Provides in-place mutable access to an occupied entry before any potential inserts
+            /// into the map.
+            #[inline]
+            pub fn and_modify<F: FnOnce(&mut V)>(self, f: F) -> Self {
+                match self {
+                    Entry::Occupied(mut inner) => {
+                        f(inner.get_mut());
+                        Entry::Occupied(inner)
+                    },
+                    Entry::Vacant(inner) => Entry::Vacant(inner),
+                }
+            }
  
  
-    /// Takes the value out of the entry, and returns it
-    pub fn remove(self) -> V {
-        unsafe { *self.inner.remove().downcast_unchecked() }
-    }
-}
+            // Additional stable methods (as of 1.60.0-nightly) that could be added:
+            // insert_entry(self, value: V) -> OccupiedEntry<'a, K, V>                     (1.59.0)
+        }
  
  
-impl<'a, A: ?Sized + UncheckedAnyExt, V: IntoBox<A>> VacantEntry<'a, A, V> {
-    /// Sets the value of the entry with the VacantEntry's key,
-    /// and returns a mutable reference to it
-    pub fn insert(self, value: V) -> &'a mut V {
-        unsafe { self.inner.insert(value.into_box()).downcast_mut_unchecked() }
-    }
-}
+        impl<'a, A: ?Sized + Downcast, V: IntoBox<A>> OccupiedEntry<'a, A, V> {
+            /// Gets a reference to the value in the entry
+            #[inline]
+            pub fn get(&self) -> &V {
+                unsafe { self.inner.get().downcast_ref_unchecked() }
+            }
  
  
-#[cfg(all(feature = "bench", test))]
-mod bench {
-    use AnyMap;
-    use test::Bencher;
+            /// Gets a mutable reference to the value in the entry
+            #[inline]
+            pub fn get_mut(&mut self) -> &mut V {
+                unsafe { self.inner.get_mut().downcast_mut_unchecked() }
+            }
  
  
-    #[bench]
-    fn insertion(b: &mut Bencher) {
-        b.iter(|| {
-            let mut data = AnyMap::new();
-            for _ in 0..100 {
-                let _ = data.insert(42);
+            /// Converts the OccupiedEntry into a mutable reference to the value in the entry
+            /// with a lifetime bound to the collection itself
+            #[inline]
+            pub fn into_mut(self) -> &'a mut V {
+                unsafe { self.inner.into_mut().downcast_mut_unchecked() }
              }
              }
-        })
-    }
  
  
-    #[bench]
-    fn get_missing(b: &mut Bencher) {
-        b.iter(|| {
-            let data = AnyMap::new();
-            for _ in 0..100 {
-                assert_eq!(data.get(), None::<&i32>);
+            /// Sets the value of the entry, and returns the entry's old value
+            #[inline]
+            pub fn insert(&mut self, value: V) -> V {
+                unsafe { *self.inner.insert(value.into_box()).downcast_unchecked() }
              }
              }
-        })
-    }
  
  
-    #[bench]
-    fn get_present(b: &mut Bencher) {
-        b.iter(|| {
-            let mut data = AnyMap::new();
-            let _ = data.insert(42);
-            // These inner loops are a feeble attempt to drown the other factors.
-            for _ in 0..100 {
-                assert_eq!(data.get(), Some(&42));
+            /// Takes the value out of the entry, and returns it
+            #[inline]
+            pub fn remove(self) -> V {
+                unsafe { *self.inner.remove().downcast_unchecked() }
              }
              }
-        })
-    }
+        }
  
  
-    macro_rules! big_benchmarks {
-        ($name:ident, $($T:ident)*) => (
-            #[bench]
-            fn $name(b: &mut Bencher) {
-                $(
-                    #[derive(Debug, PartialEq)]
-                    struct $T(&'static str);
-
-                    impl Default for $T {
-                        fn default() -> $T {
-                            $T(stringify!($T))
-                        }
-                    }
-                )*
-
-                b.iter(|| {
-                    let mut data = AnyMap::new();
-                    $(
-                        assert_eq!(data.insert($T::default()), None::<$T>);
-                    )*
-                    $(
-                        assert_eq!(data.get(), Some(&$T::default()));
-                    )*
-                })
-            }
-        );
-    }
+        impl<'a, A: ?Sized + Downcast, V: IntoBox<A>> VacantEntry<'a, A, V> {
+            /// Sets the value of the entry with the VacantEntry's key,
+            /// and returns a mutable reference to it
+            #[inline]
+            pub fn insert(self, value: V) -> &'a mut V {
+                unsafe { self.inner.insert(value.into_box()).downcast_mut_unchecked() }
+            }
+        }
  
  
-    // Pathalogical rustc/llvm case here. This takes *absurdly* long to compile (it adds something
-    // like 100 seconds), peaking at over 1GB of RAM (though -Z time-passes doesn’t pick up on
-    // that, as it’s in the middle of the “llvm modules passes [0]” that it happens and it’s all
-    // freed at the end of that pass) and adding over 700KB to the final build. (3KB per type is
-    // more than I expected, reasonably or unreasonably.) TODO determine why and get it fixed.
-    //
-    // Selected rustc -Z time-passes output, with and without this block:
-    //
-    // - item-bodies checking:   0.4 seconds without,  5.2 seconds with;
-    // - borrow checking:        0.1 seconds without, 11   seconds with;
-    // - llvm module passes [0]: 2.5 seconds without, 49   seconds with;
-    // - codegen passes [0]:     0.6 seconds without, 11.2 seconds with.
-    //
-    // Very not good.
-    big_benchmarks! {
-        insert_and_get_on_260_types,
-        A0 B0 C0 D0 E0 F0 G0 H0 I0 J0 K0 L0 M0 N0 O0 P0 Q0 R0 S0 T0 U0 V0 W0 X0 Y0 Z0
-        A1 B1 C1 D1 E1 F1 G1 H1 I1 J1 K1 L1 M1 N1 O1 P1 Q1 R1 S1 T1 U1 V1 W1 X1 Y1 Z1
-        A2 B2 C2 D2 E2 F2 G2 H2 I2 J2 K2 L2 M2 N2 O2 P2 Q2 R2 S2 T2 U2 V2 W2 X2 Y2 Z2
-        A3 B3 C3 D3 E3 F3 G3 H3 I3 J3 K3 L3 M3 N3 O3 P3 Q3 R3 S3 T3 U3 V3 W3 X3 Y3 Z3
-        A4 B4 C4 D4 E4 F4 G4 H4 I4 J4 K4 L4 M4 N4 O4 P4 Q4 R4 S4 T4 U4 V4 W4 X4 Y4 Z4
-        A5 B5 C5 D5 E5 F5 G5 H5 I5 J5 K5 L5 M5 N5 O5 P5 Q5 R5 S5 T5 U5 V5 W5 X5 Y5 Z5
-        A6 B6 C6 D6 E6 F6 G6 H6 I6 J6 K6 L6 M6 N6 O6 P6 Q6 R6 S6 T6 U6 V6 W6 X6 Y6 Z6
-        A7 B7 C7 D7 E7 F7 G7 H7 I7 J7 K7 L7 M7 N7 O7 P7 Q7 R7 S7 T7 U7 V7 W7 X7 Y7 Z7
-        A8 B8 C8 D8 E8 F8 G8 H8 I8 J8 K8 L8 M8 N8 O8 P8 Q8 R8 S8 T8 U8 V8 W8 X8 Y8 Z8
-        A9 B9 C9 D9 E9 F9 G9 H9 I9 J9 K9 L9 M9 N9 O9 P9 Q9 R9 S9 T9 U9 V9 W9 X9 Y9 Z9
-    }
+        #[cfg(test)]
+        mod tests {
+            use crate::CloneAny;
+            use super::*;
+
+            #[derive(Clone, Debug, PartialEq)] struct A(i32);
+            #[derive(Clone, Debug, PartialEq)] struct B(i32);
+            #[derive(Clone, Debug, PartialEq)] struct C(i32);
+            #[derive(Clone, Debug, PartialEq)] struct D(i32);
+            #[derive(Clone, Debug, PartialEq)] struct E(i32);
+            #[derive(Clone, Debug, PartialEq)] struct F(i32);
+            #[derive(Clone, Debug, PartialEq)] struct J(i32);
+
+            macro_rules! test_entry {
+                ($name:ident, $init:ty) => {
+                    #[test]
+                    fn $name() {
+                        let mut map = <$init>::new();
+                        assert_eq!(map.insert(A(10)), None);
+                        assert_eq!(map.insert(B(20)), None);
+                        assert_eq!(map.insert(C(30)), None);
+                        assert_eq!(map.insert(D(40)), None);
+                        assert_eq!(map.insert(E(50)), None);
+                        assert_eq!(map.insert(F(60)), None);
+
+                        // Existing key (insert)
+                        match map.entry::<A>() {
+                            Entry::Vacant(_) => unreachable!(),
+                            Entry::Occupied(mut view) => {
+                                assert_eq!(view.get(), &A(10));
+                                assert_eq!(view.insert(A(100)), A(10));
+                            }
+                        }
+                        assert_eq!(map.get::<A>().unwrap(), &A(100));
+                        assert_eq!(map.len(), 6);
+
+
+                        // Existing key (update)
+                        match map.entry::<B>() {
+                            Entry::Vacant(_) => unreachable!(),
+                            Entry::Occupied(mut view) => {
+                                let v = view.get_mut();
+                                let new_v = B(v.0 * 10);
+                                *v = new_v;
+                            }
+                        }
+                        assert_eq!(map.get::<B>().unwrap(), &B(200));
+                        assert_eq!(map.len(), 6);
  
  
-    big_benchmarks! {
-        insert_and_get_on_26_types,
-        A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
-    }
-}
  
  
-#[cfg(test)]
-mod tests {
-    use {Map, AnyMap, Entry};
-    use any::{Any, CloneAny};
-
-    #[derive(Clone, Debug, PartialEq)] struct A(i32);
-    #[derive(Clone, Debug, PartialEq)] struct B(i32);
-    #[derive(Clone, Debug, PartialEq)] struct C(i32);
-    #[derive(Clone, Debug, PartialEq)] struct D(i32);
-    #[derive(Clone, Debug, PartialEq)] struct E(i32);
-    #[derive(Clone, Debug, PartialEq)] struct F(i32);
-    #[derive(Clone, Debug, PartialEq)] struct J(i32);
-
-    macro_rules! test_entry {
-        ($name:ident, $init:ty) => {
-            #[test]
-            fn $name() {
-                let mut map = <$init>::new();
-                assert_eq!(map.insert(A(10)), None);
-                assert_eq!(map.insert(B(20)), None);
-                assert_eq!(map.insert(C(30)), None);
-                assert_eq!(map.insert(D(40)), None);
-                assert_eq!(map.insert(E(50)), None);
-                assert_eq!(map.insert(F(60)), None);
-
-                // Existing key (insert)
-                match map.entry::<A>() {
-                    Entry::Vacant(_) => unreachable!(),
-                    Entry::Occupied(mut view) => {
-                        assert_eq!(view.get(), &A(10));
-                        assert_eq!(view.insert(A(100)), A(10));
-                    }
-                }
-                assert_eq!(map.get::<A>().unwrap(), &A(100));
-                assert_eq!(map.len(), 6);
+                        // Existing key (remove)
+                        match map.entry::<C>() {
+                            Entry::Vacant(_) => unreachable!(),
+                            Entry::Occupied(view) => {
+                                assert_eq!(view.remove(), C(30));
+                            }
+                        }
+                        assert_eq!(map.get::<C>(), None);
+                        assert_eq!(map.len(), 5);
  
  
  
  
-                // Existing key (update)
-                match map.entry::<B>() {
-                    Entry::Vacant(_) => unreachable!(),
-                    Entry::Occupied(mut view) => {
-                        let v = view.get_mut();
-                        let new_v = B(v.0 * 10);
-                        *v = new_v;
+                        // Inexistent key (insert)
+                        match map.entry::<J>() {
+                            Entry::Occupied(_) => unreachable!(),
+                            Entry::Vacant(view) => {
+                                assert_eq!(*view.insert(J(1000)), J(1000));
+                            }
+                        }
+                        assert_eq!(map.get::<J>().unwrap(), &J(1000));
+                        assert_eq!(map.len(), 6);
+
+                        // Entry.or_insert on existing key
+                        map.entry::<B>().or_insert(B(71)).0 += 1;
+                        assert_eq!(map.get::<B>().unwrap(), &B(201));
+                        assert_eq!(map.len(), 6);
+
+                        // Entry.or_insert on nonexisting key
+                        map.entry::<C>().or_insert(C(300)).0 += 1;
+                        assert_eq!(map.get::<C>().unwrap(), &C(301));
+                        assert_eq!(map.len(), 7);
                      }
                  }
                      }
                  }
-                assert_eq!(map.get::<B>().unwrap(), &B(200));
-                assert_eq!(map.len(), 6);
-
+            }
  
  
-                // Existing key (remove)
-                match map.entry::<C>() {
-                    Entry::Vacant(_) => unreachable!(),
-                    Entry::Occupied(view) => {
-                        assert_eq!(view.remove(), C(30));
-                    }
-                }
-                assert_eq!(map.get::<C>(), None);
-                assert_eq!(map.len(), 5);
+            test_entry!(test_entry_any, AnyMap);
+            test_entry!(test_entry_cloneany, Map<dyn CloneAny>);
  
  
+            #[test]
+            fn test_default() {
+                let map: AnyMap = Default::default();
+                assert_eq!(map.len(), 0);
+            }
  
  
-                // Inexistent key (insert)
-                match map.entry::<J>() {
-                    Entry::Occupied(_) => unreachable!(),
-                    Entry::Vacant(view) => {
-                        assert_eq!(*view.insert(J(1000)), J(1000));
-                    }
-                }
-                assert_eq!(map.get::<J>().unwrap(), &J(1000));
-                assert_eq!(map.len(), 6);
+            #[test]
+            fn test_clone() {
+                let mut map: Map<dyn CloneAny> = Map::new();
+                let _ = map.insert(A(1));
+                let _ = map.insert(B(2));
+                let _ = map.insert(D(3));
+                let _ = map.insert(E(4));
+                let _ = map.insert(F(5));
+                let _ = map.insert(J(6));
+                let map2 = map.clone();
+                assert_eq!(map2.len(), 6);
+                assert_eq!(map2.get::<A>(), Some(&A(1)));
+                assert_eq!(map2.get::<B>(), Some(&B(2)));
+                assert_eq!(map2.get::<C>(), None);
+                assert_eq!(map2.get::<D>(), Some(&D(3)));
+                assert_eq!(map2.get::<E>(), Some(&E(4)));
+                assert_eq!(map2.get::<F>(), Some(&F(5)));
+                assert_eq!(map2.get::<J>(), Some(&J(6)));
+            }
  
  
-                // Entry.or_insert on existing key
-                map.entry::<B>().or_insert(B(71)).0 += 1;
-                assert_eq!(map.get::<B>().unwrap(), &B(201));
-                assert_eq!(map.len(), 6);
+            #[test]
+            fn test_varieties() {
+                fn assert_send<T: Send>() { }
+                fn assert_sync<T: Sync>() { }
+                fn assert_clone<T: Clone>() { }
+                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_debug::<Map<dyn Any>>();
+                assert_debug::<Map<dyn Any + Send>>();
+                assert_debug::<Map<dyn Any + Send + Sync>>();
+                assert_send::<Map<dyn CloneAny + Send>>();
+                assert_send::<Map<dyn CloneAny + Send + Sync>>();
+                assert_sync::<Map<dyn CloneAny + Send + Sync>>();
+                assert_clone::<Map<dyn CloneAny + Send>>();
+                assert_clone::<Map<dyn CloneAny + Send + Sync>>();
+                assert_clone::<Map<dyn CloneAny + Send + Sync>>();
+                assert_debug::<Map<dyn CloneAny>>();
+                assert_debug::<Map<dyn CloneAny + Send>>();
+                assert_debug::<Map<dyn CloneAny + Send + Sync>>();
+            }
  
  
-                // Entry.or_insert on nonexisting key
-                map.entry::<C>().or_insert(C(300)).0 += 1;
-                assert_eq!(map.get::<C>().unwrap(), &C(301));
-                assert_eq!(map.len(), 7);
+            #[test]
+            fn test_extend() {
+                let mut map = AnyMap::new();
+                // (vec![] for 1.36.0 compatibility; more recently, you should use [] instead.)
+                #[cfg(not(feature = "std"))]
+                use alloc::vec;
+                map.extend(vec![Box::new(123) as Box<dyn Any>, Box::new(456), Box::new(true)]);
+                assert_eq!(map.get(), Some(&456));
+                assert_eq!(map.get::<bool>(), Some(&true));
+                assert!(map.get::<Box<dyn Any>>().is_none());
              }
          }
              }
          }
-    }
+    };
+}
+
+#[cfg(feature = "std")]
+everything!(
+    "let mut data = anymap::AnyMap::new();",
+    std::collections
+);
  
  
-    test_entry!(test_entry_any, AnyMap);
-    test_entry!(test_entry_cloneany, Map<CloneAny>);
-
-    #[test]
-    fn test_clone() {
-        let mut map: Map<CloneAny> = Map::new();
-        let _ = map.insert(A(1));
-        let _ = map.insert(B(2));
-        let _ = map.insert(D(3));
-        let _ = map.insert(E(4));
-        let _ = map.insert(F(5));
-        let _ = map.insert(J(6));
-        let map2 = map.clone();
-        assert_eq!(map2.len(), 6);
-        assert_eq!(map2.get::<A>(), Some(&A(1)));
-        assert_eq!(map2.get::<B>(), Some(&B(2)));
-        assert_eq!(map2.get::<C>(), None);
-        assert_eq!(map2.get::<D>(), Some(&D(3)));
-        assert_eq!(map2.get::<E>(), Some(&E(4)));
-        assert_eq!(map2.get::<F>(), Some(&F(5)));
-        assert_eq!(map2.get::<J>(), Some(&J(6)));
+#[cfg(feature = "hashbrown")]
+/// AnyMap backed by `hashbrown`.
+///
+/// This depends on the `hashbrown` Cargo feature being enabled.
+pub mod hashbrown {
+    use crate::TypeIdHasher;
+    #[cfg(doc)]
+    use crate::any::CloneAny;
+
+    everything!(
+        "let mut data = anymap::hashbrown::AnyMap::new();",
+        hashbrown,
+        BuildHasherDefault<TypeIdHasher>
+    );
+}
+
+/// A hasher designed to eke a little more speed out, given `TypeId`’s known characteristics.
+///
+/// Specifically, this is a no-op hasher that expects to be fed a u64’s worth of
+/// randomly-distributed bits. It works well for `TypeId` (eliminating start-up time, so that my
+/// get_missing benchmark is ~30ns rather than ~900ns, and being a good deal faster after that, so
+/// that my insert_and_get_on_260_types benchmark is ~12μs instead of ~21.5μs), but will
+/// panic in debug mode and always emit zeros in release mode for any other sorts of inputs, so
+/// yeah, don’t use it! 😀
+#[derive(Default)]
+pub struct TypeIdHasher {
+    value: u64,
+}
+
+impl Hasher for TypeIdHasher {
+    #[inline]
+    fn write(&mut self, bytes: &[u8]) {
+        // This expects to receive exactly one 64-bit value, and there’s no realistic chance of
+        // that changing, but I don’t want to depend on something that isn’t expressly part of the
+        // contract for safety. But I’m OK with release builds putting everything in one bucket
+        // if it *did* change (and debug builds panicking).
+        debug_assert_eq!(bytes.len(), 8);
+        let _ = bytes.try_into()
+            .map(|array| self.value = u64::from_ne_bytes(array));
      }
  
      }
  
-    #[test]
-    fn test_varieties() {
-        fn assert_send<T: Send>() { }
-        fn assert_sync<T: Sync>() { }
-        fn assert_clone<T: Clone>() { }
-        fn assert_debug<T: ::std::fmt::Debug>() { }
-        assert_send::<Map<Any + Send>>();
-        assert_send::<Map<Any + Send + Sync>>();
-        assert_sync::<Map<Any + Sync>>();
-        assert_sync::<Map<Any + Send + Sync>>();
-        assert_debug::<Map<Any>>();
-        assert_debug::<Map<Any + Send>>();
-        assert_debug::<Map<Any + Sync>>();
-        assert_debug::<Map<Any + Send + Sync>>();
-        assert_send::<Map<CloneAny + Send>>();
-        assert_send::<Map<CloneAny + Send + Sync>>();
-        assert_sync::<Map<CloneAny + Sync>>();
-        assert_sync::<Map<CloneAny + Send + Sync>>();
-        assert_clone::<Map<CloneAny + Send>>();
-        assert_clone::<Map<CloneAny + Send + Sync>>();
-        assert_clone::<Map<CloneAny + Sync>>();
-        assert_clone::<Map<CloneAny + Send + Sync>>();
-        assert_debug::<Map<CloneAny>>();
-        assert_debug::<Map<CloneAny + Send>>();
-        assert_debug::<Map<CloneAny + Sync>>();
-        assert_debug::<Map<CloneAny + Send + Sync>>();
+    #[inline]
+    fn finish(&self) -> u64 { self.value }
+}
+
+#[test]
+fn type_id_hasher() {
+    #[cfg(not(feature = "std"))]
+    use alloc::vec::Vec;
+    use core::hash::Hash;
+    use core::any::TypeId;
+    fn verify_hashing_with(type_id: TypeId) {
+        let mut hasher = TypeIdHasher::default();
+        type_id.hash(&mut hasher);
+        // SAFETY: u64 is valid for all bit patterns.
+        assert_eq!(hasher.finish(), unsafe { core::mem::transmute::<TypeId, u64>(type_id) });
      }
      }
+    // Pick a variety of types, just to demonstrate it’s all sane. Normal, zero-sized, unsized, &c.
+    verify_hashing_with(TypeId::of::<usize>());
+    verify_hashing_with(TypeId::of::<()>());
+    verify_hashing_with(TypeId::of::<str>());
+    verify_hashing_with(TypeId::of::<&str>());
+    verify_hashing_with(TypeId::of::<Vec<u8>>());
  }
  }