fix: allow passing &dyn TimeZoneProvider in arguments (#599)

Tested the crate using `cargo semver-checks` and everything looks good,
so we could release this change in a 0.1.1 version if we want to.

Author: jedel1043

docs/architecture.md

@@ -51,7 +51,7 @@ A `TimeZoneProvider` API on a core builtin will look like the below.
 
 ```rust
 impl ZonedDateTime {
-    pub fn day_with_provider(&self, provider: &impl TimeZoneProvider) -> TemporalResult<u8> {
+    pub fn day_with_provider(&self, provider: &(impl TimeZoneProvider + ?Sized)) -> TemporalResult<u8> {
         // Code goes here.
     }
 }

provider/src/tzif.rs

@@ -1917,7 +1917,7 @@ mod tests {
             id: &str,
             before_offset: i64,
             after_offset: i64,
-            provider: &impl TimeZoneProvider,
+            provider: &(impl TimeZoneProvider + ?Sized),
         ) {
             let id = provider.get(id.as_bytes()).unwrap();
             let before_possible = provider

src/builtins/core/duration.rs

@@ -830,7 +830,7 @@ impl Duration {
         &self,
         other: &Duration,
         relative_to: Option<RelativeTo>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<Ordering> {
         if self == other {
             return Ok(Ordering::Equal);
@@ -1054,7 +1054,7 @@ impl Duration {
         &self,
         options: RoundingOptions,
         relative_to: Option<RelativeTo>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<Self> {
         // NOTE(HalidOdat): Steps 1-12 are handled before calling the function.
         //
@@ -1286,7 +1286,7 @@ impl Duration {
         &self,
         unit: Unit,
         relative_to: Option<RelativeTo>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
         // Review question what is the return type of duration.prototye.total?
     ) -> TemporalResult<FiniteF64> {
         match relative_to {

src/builtins/core/duration/normalized.rs

@@ -372,7 +372,7 @@ impl InternalDurationRecord {
         sign: Sign,
         dest_epoch_ns: i128,
         dt: &PlainDateTime,
-        time_zone: Option<(&TimeZone, &impl TimeZoneProvider)>, // ???
+        time_zone: Option<(&TimeZone, &(impl TimeZoneProvider + ?Sized))>, // ???
         options: ResolvedRoundingOptions,
     ) -> TemporalResult<NudgeRecord> {
         // NOTE: r2 may never be used...need to test.
@@ -660,7 +660,7 @@ impl InternalDurationRecord {
         dt: &PlainDateTime,
         time_zone: &TimeZone,
         options: ResolvedRoundingOptions,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<NudgeRecord> {
         let d = self.date();
         // 1.Let start be ? CalendarDateAdd(calendar, isoDateTime.[[ISODate]], duration.[[Date]], constrain).
@@ -822,7 +822,7 @@ impl InternalDurationRecord {
         sign: Sign,
         nudged_epoch_ns: i128,
         iso_date_time: &IsoDateTime,
-        time_zone: Option<(&TimeZone, &impl TimeZoneProvider)>,
+        time_zone: Option<(&TimeZone, &(impl TimeZoneProvider + ?Sized))>,
         calendar: &Calendar,
         largest_unit: Unit,
         smallest_unit: Unit,
@@ -956,7 +956,7 @@ impl InternalDurationRecord {
         &self,
         dest_epoch_ns: i128,
         dt: &PlainDateTime,
-        time_zone: Option<(&TimeZone, &impl TimeZoneProvider)>,
+        time_zone: Option<(&TimeZone, &(impl TimeZoneProvider + ?Sized))>,
         options: ResolvedRoundingOptions,
     ) -> TemporalResult<InternalDurationRecord> {
         let duration = *self;
@@ -1014,7 +1014,7 @@ impl InternalDurationRecord {
         &self,
         dest_epoch_ns: i128,
         dt: &PlainDateTime,
-        time_zone: Option<(&TimeZone, &impl TimeZoneProvider)>,
+        time_zone: Option<(&TimeZone, &(impl TimeZoneProvider + ?Sized))>,
         unit: Unit,
     ) -> TemporalResult<FiniteF64> {
         // 1. If IsCalendarUnit(unit) is true, or timeZone is not unset and unit is day, then

src/builtins/core/instant.rs

@@ -395,7 +395,7 @@ impl Instant {
     pub fn to_zoned_date_time_iso_with_provider(
         &self,
         time_zone: TimeZone,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<ZonedDateTime> {
         ZonedDateTime::new_unchecked_with_provider(*self, time_zone, Calendar::ISO, provider)
     }
@@ -409,7 +409,7 @@ impl Instant {
         &self,
         timezone: Option<TimeZone>,
         options: ToStringRoundingOptions,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<String> {
         self.to_ixdtf_writeable_with_provider(timezone, options, provider)
             .map(|x| x.write_to_string().into())
@@ -420,7 +420,7 @@ impl Instant {
         &self,
         timezone: Option<TimeZone>,
         options: ToStringRoundingOptions,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<impl Writeable + '_> {
         let resolved_options = options.resolve()?;
         let round = self.round_instant(ResolvedRoundingOptions::from_to_string_options(

src/builtins/core/now.rs

@@ -24,7 +24,7 @@ impl<H: HostHooks> Now<H> {
     pub(crate) fn system_datetime_with_provider(
         self,
         time_zone: Option<TimeZone>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<IsoDateTime> {
         let system_nanoseconds = self.host_hooks.get_system_epoch_nanoseconds()?;
         let time_zone = time_zone.unwrap_or(self.host_hooks.get_system_time_zone(provider)?);
@@ -34,7 +34,7 @@ impl<H: HostHooks> Now<H> {
     /// Converts the current [`Now`] into a [`TimeZone`].
     pub fn time_zone_with_provider(
         self,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<TimeZone> {
         self.host_hooks.get_system_time_zone(provider)
     }
@@ -50,7 +50,7 @@ impl<H: HostHooks> Now<H> {
     pub fn zoned_date_time_iso_with_provider(
         self,
         time_zone: Option<TimeZone>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<ZonedDateTime> {
         let system_nanoseconds = self.host_hooks.get_system_epoch_nanoseconds()?;
         let time_zone = time_zone.unwrap_or(self.host_hooks.get_system_time_zone(provider)?);
@@ -67,7 +67,7 @@ impl<H: HostHooks> Now<H> {
     pub fn plain_date_time_iso_with_provider(
         self,
         time_zone: Option<TimeZone>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<PlainDateTime> {
         let iso = self.system_datetime_with_provider(time_zone, provider)?;
         Ok(PlainDateTime::new_unchecked(iso, Calendar::ISO))
@@ -80,7 +80,7 @@ impl<H: HostHooks> Now<H> {
     pub fn plain_date_iso_with_provider(
         self,
         time_zone: Option<TimeZone>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<PlainDate> {
         let iso = self.system_datetime_with_provider(time_zone, provider)?;
         Ok(PlainDate::new_unchecked(iso.date, Calendar::ISO))
@@ -93,7 +93,7 @@ impl<H: HostHooks> Now<H> {
     pub fn plain_time_with_provider(
         self,
         time_zone: Option<TimeZone>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<PlainTime> {
         let iso = self.system_datetime_with_provider(time_zone, provider)?;
         Ok(PlainTime::new_unchecked(iso.time))
@@ -148,7 +148,10 @@ mod tests {
         }
 
         impl HostTimeZone for TestHooks {
-            fn get_host_time_zone(&self, _: &impl TimeZoneProvider) -> TemporalResult<TimeZone> {
+            fn get_host_time_zone(
+                &self,
+                _: &(impl TimeZoneProvider + ?Sized),
+            ) -> TemporalResult<TimeZone> {
                 Ok(self.time_zone)
             }
         }

src/builtins/core/plain_date.rs

@@ -660,7 +660,7 @@ impl PlainDate {
         &self,
         time_zone: TimeZone,
         plain_time: Option<PlainTime>,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<ZonedDateTime> {
         // NOTE (nekevss): Steps 1-4 are engine specific
         let epoch_ns = if let Some(time) = plain_time {

src/builtins/core/plain_date_time.rs

@@ -892,7 +892,7 @@ impl PlainDateTime {
         &self,
         time_zone: TimeZone,
         disambiguation: Disambiguation,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<ZonedDateTime> {
         // 6. Let epochNs be ? GetEpochNanosecondsFor(timeZone, dateTime.[[ISODateTime]], disambiguation).
         let epoch_ns = time_zone.get_epoch_nanoseconds_for(self.iso, disambiguation, provider)?;

src/builtins/core/plain_month_day.rs

@@ -353,7 +353,7 @@ impl PlainMonthDay {
     pub fn epoch_ns_for_with_provider(
         &self,
         time_zone: TimeZone,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<EpochNanoseconds> {
         // 2. Let isoDateTime be CombineISODateAndTimeRecord(temporalYearMonth.[[ISODate]], NoonTimeRecord()).
         let iso = IsoDateTime::new(self.iso, IsoTime::noon())?;

src/builtins/core/plain_year_month.rs

@@ -615,7 +615,7 @@ impl PlainYearMonth {
     pub fn epoch_ns_for_with_provider(
         &self,
         time_zone: TimeZone,
-        provider: &impl TimeZoneProvider,
+        provider: &(impl TimeZoneProvider + ?Sized),
     ) -> TemporalResult<EpochNanoseconds> {
         // 2. Let isoDateTime be CombineISODateAndTimeRecord(temporalYearMonth.[[ISODate]], NoonTimeRecord()).
         let iso = IsoDateTime::new(self.iso, IsoTime::noon())?;