Style updates (#632)

Update impl item order of `Now` and make `Temporal::now` a `const fn`

Author: Sharktheone

src/builtins/compiled/now.rs

@@ -12,6 +12,11 @@ impl<H: HostHooks> Now<H> {
         self.time_zone_with_provider(&*TZ_PROVIDER)
     }
 
+    /// Converts the current [`Now`] into an [`ZonedDateTime`] with an ISO8601 calendar.
+    pub fn zoned_date_time_iso(self, time_zone: Option<TimeZone>) -> TemporalResult<ZonedDateTime> {
+        self.zoned_date_time_iso_with_provider(time_zone, &*TZ_PROVIDER)
+    }
+
     /// Returns the current system time as a [`PlainDateTime`] with an optional
     /// [`TimeZone`].
     ///
@@ -35,9 +40,4 @@ impl<H: HostHooks> Now<H> {
     pub fn plain_time_iso(self, time_zone: Option<TimeZone>) -> TemporalResult<PlainTime> {
         self.plain_time_with_provider(time_zone, &*TZ_PROVIDER)
     }
-
-    /// Converts the current [`Now`] into an [`ZonedDateTime`] with an ISO8601 calendar.
-    pub fn zoned_date_time_iso(self, time_zone: Option<TimeZone>) -> TemporalResult<ZonedDateTime> {
-        self.zoned_date_time_iso_with_provider(time_zone, &*TZ_PROVIDER)
-    }
 }

src/builtins/core/now.rs

@@ -30,7 +30,9 @@ impl<H: HostHooks> Now<H> {
         let time_zone = time_zone.unwrap_or(self.host_hooks.get_system_time_zone(provider)?);
         time_zone.get_iso_datetime_for(&Instant::from(system_nanoseconds), provider)
     }
+}
 
+impl<H: HostHooks> Now<H> {
     /// Converts the current [`Now`] into a [`TimeZone`].
     pub fn time_zone_with_provider(
         self,
@@ -57,9 +59,7 @@ impl<H: HostHooks> Now<H> {
         let instant = Instant::from(system_nanoseconds);
         ZonedDateTime::new_unchecked_with_provider(instant, time_zone, Calendar::ISO, provider)
     }
-}
 
-impl<H: HostHooks> Now<H> {
     /// Converts `Now` into the current system [`PlainDateTime`] with an ISO8601 calendar.
     ///
     /// When `TimeZone` is `None`, the value will default to the

src/sys.rs

@@ -25,7 +25,7 @@ pub struct Temporal;
 
 impl Temporal {
     /// Get a `Now` object for the default host system.
-    pub fn now() -> Now<DefaultHostSystem> {
+    pub const fn now() -> Now<DefaultHostSystem> {
         Now::new(DefaultHostSystem)
     }
 }