Use weak-dependency

Avoid multiplying flavors of "nightly" feature

Author: zakarumych

Cargo.toml

@@ -40,13 +40,11 @@ bumpalo = "3.6.0"
 [features]
 default = ["ahash", "inline-more", "allocator-api2"]
 
-nightly-base = [] # non-public feature to use nightly features without depending on `allocator-api2`
-
-nightly = ["allocator-api2/nightly", "nightly-base", "bumpalo/allocator_api"]
+nightly = ["allocator-api2?/nightly", "bumpalo/allocator_api"]
 
 rustc-internal-api = []
 rustc-dep-of-std = [
-    "nightly-base",
+    "nightly",
     "core",
     "compiler_builtins",
     "alloc",

src/lib.rs

@@ -11,7 +11,7 @@
 
 #![no_std]
 #![cfg_attr(
-    feature = "nightly-base",
+    feature = "nightly",
     feature(
         test,
         core_intrinsics,
@@ -37,7 +37,7 @@
 )]
 #![warn(missing_docs)]
 #![warn(rust_2018_idioms)]
-#![cfg_attr(feature = "nightly-base", warn(fuzzy_provenance_casts))]
+#![cfg_attr(feature = "nightly", warn(fuzzy_provenance_casts))]
 
 #[cfg(test)]
 #[macro_use]
@@ -46,7 +46,7 @@ extern crate std;
 #[cfg_attr(test, macro_use)]
 extern crate alloc;
 
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 #[cfg(doctest)]
 doc_comment::doctest!("../README.md");
 

src/macros.rs

@@ -56,13 +56,13 @@ macro_rules! cfg_if {
 
 // Helper macro for specialization. This also helps avoid parse errors if the
 // default fn syntax for specialization changes in the future.
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 macro_rules! default_fn {
 	(#[$($a:tt)*] $($tt:tt)*) => {
         #[$($a)*] default $($tt)*
     }
 }
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 macro_rules! default_fn {
 	($($tt:tt)*) => {
         $($tt)*

src/map.rs

@@ -237,7 +237,7 @@ where
     move |x| k.equivalent(x)
 }
 
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[cfg_attr(feature = "inline-more", inline)]
 pub(crate) fn make_hash<Q, S>(hash_builder: &S, val: &Q) -> u64
 where
@@ -250,7 +250,7 @@ where
     state.finish()
 }
 
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 #[cfg_attr(feature = "inline-more", inline)]
 pub(crate) fn make_hash<Q, S>(hash_builder: &S, val: &Q) -> u64
 where
@@ -260,7 +260,7 @@ where
     hash_builder.hash_one(val)
 }
 
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[cfg_attr(feature = "inline-more", inline)]
 pub(crate) fn make_insert_hash<K, S>(hash_builder: &S, val: &K) -> u64
 where
@@ -273,7 +273,7 @@ where
     state.finish()
 }
 
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 #[cfg_attr(feature = "inline-more", inline)]
 pub(crate) fn make_insert_hash<K, S>(hash_builder: &S, val: &K) -> u64
 where
@@ -6498,13 +6498,13 @@ where
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_one(&mut self, (k, v): (K, V)) {
         self.insert(k, v);
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_reserve(&mut self, additional: usize) {
         // Keys may be already present or show multiple times in the iterator.
         // Reserve the entire hint lower bound if the map is empty.
@@ -6572,13 +6572,13 @@ where
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_one(&mut self, (k, v): (&'a K, &'a V)) {
         self.insert(*k, *v);
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_reserve(&mut self, additional: usize) {
         Extend::<(K, V)>::extend_reserve(self, additional);
     }
@@ -6632,13 +6632,13 @@ where
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_one(&mut self, &(k, v): &'a (K, V)) {
         self.insert(k, v);
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_reserve(&mut self, additional: usize) {
         Extend::<(K, V)>::extend_reserve(self, additional);
     }

src/raw/alloc.rs

@@ -4,7 +4,7 @@ pub(crate) use self::inner::{do_alloc, Allocator, Global};
 // Use unstable `allocator_api` feature.
 // This is compatible with `allocator-api2` which can be enabled or not.
 // This is used when building for `std`.
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 mod inner {
     use crate::alloc::alloc::Layout;
     pub use crate::alloc::alloc::{Allocator, Global};
@@ -25,7 +25,7 @@ mod inner {
 // this will be equivalent to the nightly case,
 // since `allocator_api2::alloc::Allocator` would be re-export of
 // `core::alloc::Allocator`.
-#[cfg(all(not(feature = "nightly-base"), feature = "allocator-api2"))]
+#[cfg(all(not(feature = "nightly"), feature = "allocator-api2"))]
 mod inner {
     use crate::alloc::alloc::Layout;
     pub use allocator_api2::alloc::{Allocator, Global};
@@ -48,7 +48,7 @@ mod inner {
 // in this crate.
 // Any crate in build-tree can enable `allocator-api2`,
 // or `nightly` without disturbing users that don't want to use it.
-#[cfg(not(any(feature = "nightly-base", feature = "allocator-api2")))]
+#[cfg(not(any(feature = "nightly", feature = "allocator-api2")))]
 mod inner {
     use crate::alloc::alloc::{alloc, dealloc, Layout};
     use core::ptr::NonNull;

src/raw/bitmask.rs

@@ -1,5 +1,5 @@
 use super::imp::{BitMaskWord, BITMASK_MASK, BITMASK_STRIDE};
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 use core::intrinsics;
 
 /// A bit mask which contains the result of a `Match` operation on a `Group` and
@@ -64,12 +64,12 @@ impl BitMask {
     /// Returns the first set bit in the `BitMask`, if there is one. The
     /// bitmask must not be empty.
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     pub(crate) unsafe fn lowest_set_bit_nonzero(self) -> usize {
         intrinsics::cttz_nonzero(self.0) as usize / BITMASK_STRIDE
     }
     #[inline]
-    #[cfg(not(feature = "nightly-base"))]
+    #[cfg(not(feature = "nightly"))]
     pub(crate) unsafe fn lowest_set_bit_nonzero(self) -> usize {
         self.trailing_zeros()
     }

src/raw/mod.rs

@@ -42,25 +42,25 @@ use self::imp::Group;
 
 // Branch prediction hint. This is currently only available on nightly but it
 // consistently improves performance by 10-15%.
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 use core::intrinsics::{likely, unlikely};
 
 // On stable we can use #[cold] to get a equivalent effect: this attributes
 // suggests that the function is unlikely to be called
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[inline]
 #[cold]
 fn cold() {}
 
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[inline]
 fn likely(b: bool) -> bool {
     if !b {
         cold();
     }
     b
 }
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[inline]
 fn unlikely(b: bool) -> bool {
     if b {
@@ -70,10 +70,10 @@ fn unlikely(b: bool) -> bool {
 }
 
 // Use strict provenance functions if available.
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 use core::ptr::invalid_mut;
 // Implement it with a cast otherwise.
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 #[inline(always)]
 fn invalid_mut<T>(addr: usize) -> *mut T {
     addr as *mut T
@@ -936,7 +936,7 @@ impl<T, A: Allocator + Clone> RawTable<T, A> {
 
     /// Returns pointer to start of data table.
     #[inline]
-    #[cfg(any(feature = "raw", feature = "nightly-base"))]
+    #[cfg(any(feature = "raw", feature = "nightly"))]
     pub unsafe fn data_start(&self) -> NonNull<T> {
         NonNull::new_unchecked(self.data_end().as_ptr().wrapping_sub(self.buckets()))
     }
@@ -2562,7 +2562,7 @@ impl<T: Clone, A: Allocator + Clone> RawTableClone for RawTable<T, A> {
         }
     }
 }
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 impl<T: Copy, A: Allocator + Clone> RawTableClone for RawTable<T, A> {
     #[cfg_attr(feature = "inline-more", inline)]
     unsafe fn clone_from_spec(&mut self, source: &Self) {
@@ -2674,7 +2674,7 @@ impl<T, A: Allocator + Clone + Default> Default for RawTable<T, A> {
     }
 }
 
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 unsafe impl<#[may_dangle] T, A: Allocator + Clone> Drop for RawTable<T, A> {
     #[cfg_attr(feature = "inline-more", inline)]
     fn drop(&mut self) {
@@ -2686,7 +2686,7 @@ unsafe impl<#[may_dangle] T, A: Allocator + Clone> Drop for RawTable<T, A> {
         }
     }
 }
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 impl<T, A: Allocator + Clone> Drop for RawTable<T, A> {
     #[cfg_attr(feature = "inline-more", inline)]
     fn drop(&mut self) {
@@ -3081,7 +3081,7 @@ where
 {
 }
 
-#[cfg(feature = "nightly-base")]
+#[cfg(feature = "nightly")]
 unsafe impl<#[may_dangle] T, A: Allocator + Clone> Drop for RawIntoIter<T, A> {
     #[cfg_attr(feature = "inline-more", inline)]
     fn drop(&mut self) {
@@ -3096,7 +3096,7 @@ unsafe impl<#[may_dangle] T, A: Allocator + Clone> Drop for RawIntoIter<T, A> {
         }
     }
 }
-#[cfg(not(feature = "nightly-base"))]
+#[cfg(not(feature = "nightly"))]
 impl<T, A: Allocator + Clone> Drop for RawIntoIter<T, A> {
     #[cfg_attr(feature = "inline-more", inline)]
     fn drop(&mut self) {

src/set.rs

@@ -1354,13 +1354,13 @@ where
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_one(&mut self, k: T) {
         self.map.insert(k, ());
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_reserve(&mut self, additional: usize) {
         Extend::<(T, ())>::extend_reserve(&mut self.map, additional);
     }
@@ -1378,13 +1378,13 @@ where
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_one(&mut self, k: &'a T) {
         self.map.insert(*k, ());
     }
 
     #[inline]
-    #[cfg(feature = "nightly-base")]
+    #[cfg(feature = "nightly")]
     fn extend_reserve(&mut self, additional: usize) {
         Extend::<(T, ())>::extend_reserve(&mut self.map, additional);
     }