Clock::currentTime: Avoid calling unit.convert when the asked TimeUnit is MILLISECONDS (zio#8358)

guizmaii · web-flow · commit 4e9809cc9cd0 · 2023-08-29T06:53:41.000-07:00
diff --git a/core/shared/src/main/scala/zio/Clock.scala b/core/shared/src/main/scala/zio/Clock.scala
@@ -119,7 +119,8 @@ object Clock extends ClockPlatformSpecific with Serializable {
               inst.getEpochSecond * 1000000000 + inst.getNano
             case TimeUnit.MICROSECONDS =>
               inst.getEpochSecond * 1000000 + inst.getNano / 1000
-            case _ => unit.convert(inst.toEpochMilli, TimeUnit.MILLISECONDS)
+            case TimeUnit.MILLISECONDS => inst.toEpochMilli
+            case _                     => unit.convert(inst.toEpochMilli, TimeUnit.MILLISECONDS)
           }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,8 @@ object Clock extends ClockPlatformSpecific with Serializable {`
`119`	`119`	`inst.getEpochSecond * 1000000000 + inst.getNano`
`120`	`120`	`case TimeUnit.MICROSECONDS =>`
`121`	`121`	`inst.getEpochSecond * 1000000 + inst.getNano / 1000`
`122`		`- case _ => unit.convert(inst.toEpochMilli, TimeUnit.MILLISECONDS)`
	`122`	`+ case TimeUnit.MILLISECONDS => inst.toEpochMilli`
	`123`	`+ case _ => unit.convert(inst.toEpochMilli, TimeUnit.MILLISECONDS)`
`123`	`124`	`}`
`124`	`125`	`}`
`125`	`126`