Add getCurrentNtpTimeMs

ameliariely commented 5 years ago

Add a method that will only ever return NTP synced time, or null.

We currently have two options for getting time:

getCurrentTimeMs() always returns a time as a Long. It falls back to the local clock if no ntp sync has occurred.
getCurrentTime() always returns a time as a KronosTime. The KronosTime.posixTimeMs is equal to the value of getCurrentTimeMs(). You can discover whether we fell back to the fallback clock by checking timeSinceLastNtpSyncMs == null. This works, but seems unnecessarily complicated.

ameliariely commented 5 years ago

👀 @ghackett @amphora001 @artem-zinnatullin

ameliariely commented 5 years ago

cc @Reflejo

ghackett commented 5 years ago

👍

amphora001 commented 5 years ago

👍

lyft / Kronos-Android