You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.
Follow up from #8576 where both @jfirebaugh as @ivovandongen suggested to look into using an object for duration instead of a long. This allows us to centralize converting logic and avoid any future unit conversion bugs due to type safety.
If we force usage of this in the public API, this would be a SemVer change. We can also opt to gradually migrate to it (leave long for public api and do a conversion to a Duration underneath until next major release).
The text was updated successfully, but these errors were encountered:
After some internal discussion, we are going to punt on using long for now. This matches the end deveoper expectations the most as Android platform code also uses long for duration.
Follow up from #8576 where both @jfirebaugh as @ivovandongen suggested to look into using an object for duration instead of a long. This allows us to centralize converting logic and avoid any future unit conversion bugs due to type safety.
If we force usage of this in the public API, this would be a SemVer change. We can also opt to gradually migrate to it (leave long for public api and do a conversion to a Duration underneath until next major release).
The text was updated successfully, but these errors were encountered: