Change std to core where possible

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

index a53a2d8f1e88ca0d711da90fe7a7f4889fd6996a..966aa5a6a2312400cb4f6cf6cf71df416aba230e 100644 (file)
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1,9 +1,11 @@
-//! 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.
  
  #![warn(missing_docs, unused_results)]
  
-use std::any::{Any, TypeId};
-use std::marker::PhantomData;
+use core::any::{Any, TypeId};
+use core::marker::PhantomData;
  
  use raw::RawMap;
  use any::{UncheckedAnyExt, IntoBox};
@@ -93,14 +95,29 @@ 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
-/// 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`.
-/// - You can add on `+ Send` or `+ Send + Sync` (e.g. `Map<dyn Any + Send>`) to add those bounds.
+/// - 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
-/// # use anymap::AnyMap;
-/// let mut data = AnyMap::new();
+/// let mut data = anymap::AnyMap::new();
  /// assert_eq!(data.get(), None::<&i32>);
  /// data.insert(42i32);
  /// assert_eq!(data.get(), Some(&42i32));
@@ -135,7 +152,7 @@ impl<A: ?Sized + UncheckedAnyExt> Clone for Map<A> where Box<A>: Clone {
      }
  }
  
-/// The most common type of `Map`: just using `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.
@@ -219,10 +236,10 @@ impl<A: ?Sized + UncheckedAnyExt> AsMut<RawMap<A>> for Map<A> {
      }
  }
  
-impl<A: ?Sized + UncheckedAnyExt> Into<RawMap<A>> for Map<A> {
+impl<A: ?Sized + UncheckedAnyExt> From<Map<A>> for RawMap<A> {
      #[inline]
-    fn into(self) -> RawMap<A> {
-        self.raw
+    fn from(map: Map<A>) -> RawMap<A> {
+        map.raw
      }
  }
  
@@ -427,7 +444,7 @@ mod tests {
          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>>();