-
Notifications
You must be signed in to change notification settings - Fork 1.9k
java.time support for Flow #1165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Why you so like java.time? Why not just Longs? Android has problems with this package |
Android doesnnot have problems with it. They're only available on certain API levels, similar to most of the APIs that are exposed. Longs are in general terrible types for representing anything since it carries no semantic information. Nothing prevents you from adding milliseconds to seconds and interpreting it as hours. |
Equals to
Coroutines should work everywhere without limiting its support to android users. |
All you've done is emphasize my criticisms. |
We're working on adding durations support to Kotlin stdlib and we will support Kotlin duration types in coroutines library when they become available. There is no plan to direcly support Java's durations, but eventually there will be converters. |
It was fixed by #1834 |
Consider to overload
Flow
operators with milliseconds parameters usingDuration
instead.The text was updated successfully, but these errors were encountered: