Add two more Duration extensions. (#145)

Author: JonathanLennox

src/main/kotlin/org/jitsi/utils/Duration.kt

@@ -141,15 +141,15 @@ fun durationOfDoubleSeconds(duration: Double): Duration {
     val secs = floor(duration)
     val ns = round((duration - secs) * 1e9)
     require(secs >= Long.MIN_VALUE.toDouble() && secs <= Long.MAX_VALUE.toDouble())
-    /* For reasons I don't understand, the basic Duration(secs, ns) constructor is private. */
-    return Duration.ofSeconds(secs.toLong()).plusNanos(ns.toLong())
+    return Duration.ofSeconds(secs.toLong(), ns.toLong())
 }
 
 val MIN_DURATION: Duration = Duration.ofSeconds(Long.MIN_VALUE, 0)
 
 val MAX_DURATION: Duration = Duration.ofSeconds(Long.MAX_VALUE, 999_999_999)
 
 fun Duration.isFinite() = this != MIN_DURATION && this != MAX_DURATION
+fun Duration.isInfinite() = !isFinite()
 
 operator fun Duration.times(other: Double): Duration = durationOfDoubleSeconds((toDouble() * other))
 operator fun Double.times(other: Duration): Duration = durationOfDoubleSeconds(other.toDouble() * this)
@@ -160,6 +160,8 @@ operator fun Duration.div(other: Long): Duration = this.dividedBy(other)
 
 operator fun Duration.unaryMinus(): Duration = this.negated()
 
+fun abs(duration: Duration) = if (duration >= Duration.ZERO) duration else -duration
+
 fun <T> Iterable<T>.sumOf(selector: (T) -> Duration): Duration {
     var sum: Duration = Duration.ZERO
     for (element in this) {

src/test/kotlin/org/jitsi/utils/DurationTest.kt

@@ -73,6 +73,11 @@ class DurationTest : ShouldSpec() {
             2200.micros.toDoubleMillis() shouldBe (2.2 plusOrMinus 1e-9)
             (-(2200.micros)).toDoubleMillis() shouldBe (-2.2 plusOrMinus 1e-9)
 
+            abs(2200.micros) shouldBe 2200.micros
+            abs(-(2200.micros)) shouldBe 2200.micros
+            abs((-2200).micros) shouldBe 2200.micros
+            abs(Duration.ZERO) shouldBe Duration.ZERO
+
             durationOfDoubleSeconds(2.2) shouldBe 2200.ms
 
             durationOfDoubleSeconds(Long.MAX_VALUE.toDouble()).seconds shouldBe Long.MAX_VALUE
@@ -86,6 +91,10 @@ class DurationTest : ShouldSpec() {
             MAX_DURATION.isFinite() shouldBe false
             MIN_DURATION.isFinite() shouldBe false
 
+            2200.micros.isInfinite() shouldBe false
+            MAX_DURATION.isInfinite() shouldBe true
+            MIN_DURATION.isInfinite() shouldBe true
+
             max(2200.ms, 3000.ms) shouldBe 3000.ms
             max(2200.ms, MAX_DURATION) shouldBe MAX_DURATION
             max(2200.ms, MIN_DURATION) shouldBe 2200.ms