-
Notifications
You must be signed in to change notification settings - Fork 0
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
chrono 0.5 Duration #59
Comments
https://github.com/chronotope/chrono/releases/tag/v0.4.30 Oh damn seems like they changed the
|
So |
omg https://github.com/chronotope/chrono/releases/tag/v0.4.31 in v0.4.31 they added rusty-endsong-parser/src/lib.rs Lines 94 to 95 in a31db62
I'm gonna be able to get rid of this line :D |
hmm at least the |
Heh, Clippy now has a lint for it, I guess? :D |
See chronotope/chrono#1137
When they do this, I'll have to probably rename
chrono::Duration
to whatever they decide on (currentlyTimeDelta
), but I'll finally be able to use+=
(no longer necessary tbh), but they implementiter::Sum
!!!https://github.com/chronotope/chrono/blob/f5ea3ec216d8c34cd0346bc8b105287f9f641d2e/src/time_delta.rs#L392-L406
I won't have to use
.fold(...)
xd but a more concise.sum()
instead :De.g. here
rusty-endsong-parser/src/gather.rs
Lines 239 to 246 in 9b2d550
The text was updated successfully, but these errors were encountered: