-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Time function improvements (fix #1912) #2863
base: master
Are you sure you want to change the base?
Conversation
|
a958727
to
3d3fef0
Compare
We can also work around the lack of a |
Er, this is fraught. For example, if I try to use |
ab0b905
to
0daf027
Compare
abbb192
to
821b125
Compare
Now with some boundary tests (which found bugs). |
95aca4e
to
880457b
Compare
I've left resolved comments to explain some things, mainly the range validation checks for all the fields of broken down time (which were wrong in an earlier version). |
Well, I think we should merge this in time for 1.7, but I'm clearing the milestone anyways. |
Ok, we should allow datetimes before 1970, so I'm making the range EDIT: Yes, so let's go to |
3d68387
to
fa3640e
Compare
mmm was this not merged before 1.7 ? @nicowilliams |
@itchyny thought it was too risky for 1.7. I intend to do a new release soon with this and a few other things. |
fa3640e
to
2050375
Compare
mktime/0
returns GMT time instead of local time #2865.mktime
preserve any fractional seconds part.timelocal
andtimegm
.