sync: reorder const_new before new_with (#6392)

tokio/src/sync/once_cell.rs

@@ -132,6 +132,47 @@ impl<T> OnceCell<T> {
         }
     }
 
+    /// Creates a new empty `OnceCell` instance.
+    ///
+    /// Equivalent to `OnceCell::new`, except that it can be used in static
+    /// variables.
+    ///
+    /// When using the `tracing` [unstable feature], a `OnceCell` created with
+    /// `const_new` will not be instrumented. As such, it will not be visible
+    /// in [`tokio-console`]. Instead, [`OnceCell::new`] should be used to
+    /// create an instrumented object if that is needed.
+    ///
+    /// # Example
+    ///
+    /// ```
+    /// use tokio::sync::OnceCell;
+    ///
+    /// static ONCE: OnceCell<u32> = OnceCell::const_new();
+    ///
+    /// async fn get_global_integer() -> &'static u32 {
+    ///     ONCE.get_or_init(|| async {
+    ///         1 + 1
+    ///     }).await
+    /// }
+    ///
+    /// #[tokio::main]
+    /// async fn main() {
+    ///     let result = get_global_integer().await;
+    ///     assert_eq!(*result, 2);
+    /// }
+    /// ```
+    ///
+    /// [`tokio-console`]: https://github.com/tokio-rs/console
+    /// [unstable feature]: crate#unstable-features
+    #[cfg(not(all(loom, test)))]
+    pub const fn const_new() -> Self {
+        OnceCell {
+            value_set: AtomicBool::new(false),
+            value: UnsafeCell::new(MaybeUninit::uninit()),
+            semaphore: Semaphore::const_new(1),
+        }
+    }
+
     /// Creates a new `OnceCell` that contains the provided value, if any.
     ///
     /// If the `Option` is `None`, this is equivalent to `OnceCell::new`.
@@ -187,47 +228,6 @@ impl<T> OnceCell<T> {
         }
     }
 
-    /// Creates a new empty `OnceCell` instance.
-    ///
-    /// Equivalent to `OnceCell::new`, except that it can be used in static
-    /// variables.
-    ///
-    /// When using the `tracing` [unstable feature], a `OnceCell` created with
-    /// `const_new` will not be instrumented. As such, it will not be visible
-    /// in [`tokio-console`]. Instead, [`OnceCell::new`] should be used to
-    /// create an instrumented object if that is needed.
-    ///
-    /// # Example
-    ///
-    /// ```
-    /// use tokio::sync::OnceCell;
-    ///
-    /// static ONCE: OnceCell<u32> = OnceCell::const_new();
-    ///
-    /// async fn get_global_integer() -> &'static u32 {
-    ///     ONCE.get_or_init(|| async {
-    ///         1 + 1
-    ///     }).await
-    /// }
-    ///
-    /// #[tokio::main]
-    /// async fn main() {
-    ///     let result = get_global_integer().await;
-    ///     assert_eq!(*result, 2);
-    /// }
-    /// ```
-    ///
-    /// [`tokio-console`]: https://github.com/tokio-rs/console
-    /// [unstable feature]: crate#unstable-features
-    #[cfg(not(all(loom, test)))]
-    pub const fn const_new() -> Self {
-        OnceCell {
-            value_set: AtomicBool::new(false),
-            value: UnsafeCell::new(MaybeUninit::uninit()),
-            semaphore: Semaphore::const_new(1),
-        }
-    }
-
     /// Returns `true` if the `OnceCell` currently contains a value, and `false`
     /// otherwise.
     pub fn initialized(&self) -> bool {