Refactor native TimeZoneImpl step 2

ilya-g · ilya-g · commit 53f4425a74a5 · 2021-07-02T18:08:52.000+03:00
Remove the interface and merge it into RegionTimeZone class
diff --git a/core/darwin/src/TimeZoneNative.kt b/core/darwin/src/TimeZoneNative.kt
@@ -26,17 +26,17 @@ private fun systemDateByLocalDate(zone: NSTimeZone, localDate: NSDate): NSDate?
     return iso8601.dateFromComponents(dateComponents)
 }
 
-internal actual class PlatformTimeZoneImpl(private val value: NSTimeZone, override val id: String): TimeZoneImpl {
+internal actual class RegionTimeZone(private val value: NSTimeZone, actual override val id: String): TimeZone() {
     actual companion object {
-        actual fun of(zoneId: String): PlatformTimeZoneImpl {
+        actual fun of(zoneId: String): RegionTimeZone {
             val abbreviations = NSTimeZone.abbreviationDictionary
             val trueZoneId = abbreviations[zoneId] as String? ?: zoneId
             val zone = NSTimeZone.timeZoneWithName(trueZoneId)
                 ?: throw IllegalTimeZoneException("No timezone found with zone ID '$zoneId'")
-            return PlatformTimeZoneImpl(zone, zoneId)
+            return RegionTimeZone(zone, zoneId)
         }
 
-        actual fun currentSystemDefault(): PlatformTimeZoneImpl {
+        actual fun currentSystemDefault(): RegionTimeZone {
             /* The framework has its own cache of the system timezone. Calls to
             [NSTimeZone systemTimeZone] do not reflect changes to the system timezone
             and instead just return the cached value. Thus, to acquire the current
@@ -86,7 +86,7 @@ internal actual class PlatformTimeZoneImpl(private val value: NSTimeZone, overri
             */
             NSTimeZone.resetSystemTimeZone()
             val zone = NSTimeZone.systemTimeZone
-            return PlatformTimeZoneImpl(zone, zone.name)
+            return RegionTimeZone(zone, zone.name)
         }
 
         actual val availableZoneIds: Set<String>
@@ -110,7 +110,7 @@ internal actual class PlatformTimeZoneImpl(private val value: NSTimeZone, overri
             }
     }
 
-    override fun atStartOfDay(date: LocalDate): Instant {
+    actual override fun atStartOfDay(date: LocalDate): Instant {
         val ldt = LocalDateTime(date, LocalTime.MIN)
         val epochSeconds = ldt.toEpochSecond(UtcOffset.ZERO)
         // timezone
@@ -132,15 +132,15 @@ internal actual class PlatformTimeZoneImpl(private val value: NSTimeZone, overri
         return Instant(midnight.timeIntervalSince1970.toLong(), 0)
     }
 
-    override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime {
+    actual override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime {
         val epochSeconds = dateTime.toEpochSecond(UtcOffset.ZERO)
         var offset = preferred?.totalSeconds ?: Int.MAX_VALUE
         val transitionDuration = run {
             /* a date in an unspecified timezone, defined by the number of seconds since
                the start of the epoch in *that* unspecified timezone */
             val date = dateWithTimeIntervalSince1970Saturating(epochSeconds)
             val newDate = systemDateByLocalDate(value, date)
-                ?: throw RuntimeException("Unable to acquire the offset at $dateTime for zone ${this@PlatformTimeZoneImpl}")
+                ?: throw RuntimeException("Unable to acquire the offset at $dateTime for zone ${this@RegionTimeZone}")
             // we now know the offset of that timezone at this time.
             offset = value.secondsFromGMTForDate(newDate).toInt()
             /* `dateFromComponents` automatically corrects the date to avoid gaps. We
@@ -155,23 +155,14 @@ internal actual class PlatformTimeZoneImpl(private val value: NSTimeZone, overri
         } catch (e: ArithmeticException) {
             throw RuntimeException("Anomalously long timezone transition gap reported", e)
         }
-        return ZonedDateTime(correctedDateTime, RegionTimeZone(this), UtcOffset.ofSeconds(offset))
+        return ZonedDateTime(correctedDateTime, this@RegionTimeZone, UtcOffset.ofSeconds(offset))
     }
 
-    override fun offsetAt(instant: Instant): UtcOffset {
+    actual override fun offsetAtImpl(instant: Instant): UtcOffset {
         val date = dateWithTimeIntervalSince1970Saturating(instant.epochSeconds)
         return UtcOffset.ofSeconds(value.secondsFromGMTForDate(date).toInt())
     }
 
-    // org.threeten.bp.ZoneId#equals
-    override fun equals(other: Any?): Boolean =
-        this === other || other is PlatformTimeZoneImpl && this.id == other.id
-
-    // org.threeten.bp.ZoneId#hashCode
-    override fun hashCode(): Int = id.hashCode()
-
-    // org.threeten.bp.ZoneId#toString
-    override fun toString(): String = id
 }
 
 internal actual fun currentTime(): Instant = NSDate.date().toKotlinInstant()
diff --git a/core/native/cinterop_actuals/TimeZoneNative.kt b/core/native/cinterop_actuals/TimeZoneNative.kt
@@ -8,23 +8,23 @@ import kotlinx.datetime.internal.*
 import kotlinx.cinterop.*
 import platform.posix.free
 
-internal actual class PlatformTimeZoneImpl(private val tzid: TZID, override val id: String): TimeZoneImpl {
+internal actual class RegionTimeZone(private val tzid: TZID, actual override val id: String): TimeZone() {
     actual companion object {
-        actual fun of(zoneId: String): PlatformTimeZoneImpl {
+        actual fun of(zoneId: String): RegionTimeZone {
             val tzid = timezone_by_name(zoneId)
             if (tzid == TZID_INVALID) {
                 throw IllegalTimeZoneException("No timezone found with zone ID '$zoneId'")
             }
-            return PlatformTimeZoneImpl(tzid, zoneId)
+            return RegionTimeZone(tzid, zoneId)
         }
 
-        actual fun currentSystemDefault(): PlatformTimeZoneImpl = memScoped {
+        actual fun currentSystemDefault(): RegionTimeZone = memScoped {
             val tzid = alloc<TZIDVar>()
             val string = get_system_timezone(tzid.ptr)
                 ?: throw RuntimeException("Failed to get the system timezone.")
             val kotlinString = string.toKString()
             free(string)
-            PlatformTimeZoneImpl(tzid.value, kotlinString)
+            RegionTimeZone(tzid.value, kotlinString)
         }
 
         actual val availableZoneIds: Set<String>
@@ -45,7 +45,7 @@ internal actual class PlatformTimeZoneImpl(private val tzid: TZID, override val
             }
     }
 
-    override fun atStartOfDay(date: LocalDate): Instant = memScoped {
+    actual override fun atStartOfDay(date: LocalDate): Instant = memScoped {
         val ldt = LocalDateTime(date, LocalTime.MIN)
         val epochSeconds = ldt.toEpochSecond(UtcOffset.ZERO)
         val midnightInstantSeconds = at_start_of_day(tzid, epochSeconds)
@@ -55,13 +55,13 @@ internal actual class PlatformTimeZoneImpl(private val tzid: TZID, override val
         Instant(midnightInstantSeconds, 0)
     }
 
-    override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime = memScoped {
+    actual override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime = memScoped {
         val epochSeconds = dateTime.toEpochSecond(UtcOffset.ZERO)
         val offset = alloc<IntVar>()
         offset.value = preferred?.totalSeconds ?: Int.MAX_VALUE
         val transitionDuration = offset_at_datetime(tzid, epochSeconds, offset.ptr)
         if (offset.value == Int.MAX_VALUE) {
-            throw RuntimeException("Unable to acquire the offset at $dateTime for zone ${this@PlatformTimeZoneImpl}")
+            throw RuntimeException("Unable to acquire the offset at $dateTime for zone ${this@RegionTimeZone}")
         }
         val correctedDateTime = try {
             dateTime.plusSeconds(transitionDuration)
@@ -70,26 +70,17 @@ internal actual class PlatformTimeZoneImpl(private val tzid: TZID, override val
         } catch (e: ArithmeticException) {
             throw RuntimeException("Anomalously long timezone transition gap reported", e)
         }
-        ZonedDateTime(correctedDateTime, RegionTimeZone(this@PlatformTimeZoneImpl), UtcOffset.ofSeconds(offset.value))
+        ZonedDateTime(correctedDateTime, this@RegionTimeZone, UtcOffset.ofSeconds(offset.value))
     }
 
-    override fun offsetAt(instant: Instant): UtcOffset {
+    actual override fun offsetAtImpl(instant: Instant): UtcOffset {
         val offset = offset_at_instant(tzid, instant.epochSeconds)
         if (offset == Int.MAX_VALUE) {
             throw RuntimeException("Unable to acquire the offset at instant $instant for zone $this")
         }
         return UtcOffset.ofSeconds(offset)
     }
 
-    // org.threeten.bp.ZoneId#equals
-    override fun equals(other: Any?): Boolean =
-        this === other || other is PlatformTimeZoneImpl && this.id == other.id
-
-    // org.threeten.bp.ZoneId#hashCode
-    override fun hashCode(): Int = id.hashCode()
-
-    // org.threeten.bp.ZoneId#toString
-    override fun toString(): String = id
 }
 
 internal actual fun currentTime(): Instant = memScoped {
diff --git a/core/native/src/TimeZone.kt b/core/native/src/TimeZone.kt
@@ -18,7 +18,7 @@ public actual open class TimeZone internal constructor() {
 
         public actual fun currentSystemDefault(): TimeZone =
             // TODO: probably check if currentSystemDefault name is parseable as FixedOffsetTimeZone?
-            RegionTimeZone(PlatformTimeZoneImpl.currentSystemDefault())
+            RegionTimeZone.currentSystemDefault()
 
         public actual val UTC: FixedOffsetTimeZone = UtcOffset.ZERO.asTimeZone()
 
@@ -58,11 +58,11 @@ public actual open class TimeZone internal constructor() {
             } catch (e: DateTimeFormatException) {
                 throw IllegalTimeZoneException(e)
             }
-            return RegionTimeZone(PlatformTimeZoneImpl.of(zoneId))
+            return RegionTimeZone.of(zoneId)
         }
 
         public actual val availableZoneIds: Set<String>
-            get() = PlatformTimeZoneImpl.availableZoneIds
+            get() = RegionTimeZone.availableZoneIds
     }
 
     public actual open val id: String
@@ -94,15 +94,17 @@ public actual open class TimeZone internal constructor() {
     override fun toString(): String = id
 }
 
-internal class RegionTimeZone internal constructor(internal val value: TimeZoneImpl) : TimeZone() {
-    override val id: String get() = value.id
+internal expect class RegionTimeZone : TimeZone {
+    override val id: String
+    override fun atStartOfDay(date: LocalDate): Instant
+    override fun offsetAtImpl(instant: Instant): UtcOffset
+    override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime
 
-    override fun atStartOfDay(date: LocalDate): Instant = value.atStartOfDay(date)
-
-    override fun offsetAtImpl(instant: Instant): UtcOffset = value.offsetAt(instant)
-
-    override fun atZone(dateTime: LocalDateTime, preferred: UtcOffset?): ZonedDateTime =
-        value.atZone(dateTime, preferred)
+    companion object {
+        fun of(zoneId: String): RegionTimeZone
+        fun currentSystemDefault(): RegionTimeZone
+        val availableZoneIds: Set<String>
+    }
 }
 
 
diff --git a/core/native/src/TimeZoneImpl.kt b/core/native/src/TimeZoneImpl.kt
