Don't convert Long to Int before adding time value to the calendar.

kizitonwose · Mar 24, 2020 · 6b4d9d3 · 6b4d9d3
1 parent 7dfcfee
commit 6b4d9d3
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/time/src/main/kotlin/com/kizitonwose/time/Extensions.kt b/time/src/main/kotlin/com/kizitonwose/time/Extensions.kt
@@ -9,16 +9,12 @@ import kotlin.concurrent.scheduleAtFixedRate
  */
 
 //region Calendar
-operator fun Calendar.plus(other: Interval<TimeUnit>): Calendar {
-    val newCalendar = clone() as Calendar
-    newCalendar.add(Calendar.MILLISECOND, other.inMilliseconds.longValue.toInt())
-    return newCalendar
+operator fun Calendar.plus(other: Interval<TimeUnit>): Calendar = (clone() as Calendar).apply {
+    timeInMillis += other.inMilliseconds.longValue
 }
 
-operator fun Calendar.minus(other: Interval<TimeUnit>): Calendar {
-    val newCalendar = clone() as Calendar
-    newCalendar.add(Calendar.MILLISECOND, -other.inMilliseconds.longValue.toInt())
-    return newCalendar
+operator fun Calendar.minus(other: Interval<TimeUnit>): Calendar = (clone() as Calendar).apply {
+    timeInMillis -= other.inMilliseconds.longValue
 }
 //endregion