Thread overview | ||||||
---|---|---|---|---|---|---|
|
March 21, 2011 [Issue 5761] New: std.datetime: Date.this(int day) conversion fails for Dec 30 of leap years | ||||
---|---|---|---|---|
| ||||
http://d.puremagic.com/issues/show_bug.cgi?id=5761 Summary: std.datetime: Date.this(int day) conversion fails for Dec 30 of leap years Product: D Version: D2 Platform: All OS/Version: All Status: NEW Severity: normal Priority: P2 Component: Phobos AssignedTo: jmdavisProg@gmx.com ReportedBy: thecybershadow@gmail.com --- Comment #0 from Vladimir <thecybershadow@gmail.com> 2011-03-21 04:59:29 PDT --- import std.datetime; void main() { assert(SysTime(Date(2008, 12, 30)).day == 30); } -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
March 24, 2011 [Issue 5761] std.datetime: Date.this(int day) conversion fails for Dec 30 of leap years | ||||
---|---|---|---|---|
| ||||
Posted in reply to Vladimir | http://d.puremagic.com/issues/show_bug.cgi?id=5761 --- Comment #1 from Jonathan M Davis <jmdavisProg@gmx.com> 2011-03-23 23:32:08 PDT --- Now, _that_ was a nasty little math bug. Well, I now have a fix, and if you need it, the code can be gotten here: https://github.com/jmdavis/phobos/tree/datetime I'm revamping std.datetime's unit tests at the moment, and I probably won't create a pull request until that's done. Also, with these changes, the unit tests currently fail on Windows due to bug# 5781, so that's going to need to be fixed before a pull request is made. However, the changes are posted in the datetime branch of my github repository if you need them before they actually get merged into Phobos proper. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
March 24, 2011 [Issue 5761] std.datetime: Date.this(int day) conversion fails for Dec 30 of leap years | ||||
---|---|---|---|---|
| ||||
Posted in reply to Vladimir | http://d.puremagic.com/issues/show_bug.cgi?id=5761 --- Comment #2 from Vladimir <thecybershadow@gmail.com> 2011-03-24 13:19:59 PDT --- Thanks, the fix worked :) Feel free to close the issue when the fix is in. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
May 05, 2011 [Issue 5761] std.datetime: Date.this(int day) conversion fails for Dec 30 of leap years | ||||
---|---|---|---|---|
| ||||
Posted in reply to Vladimir | http://d.puremagic.com/issues/show_bug.cgi?id=5761 Jonathan M Davis <jmdavisProg@gmx.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #3 from Jonathan M Davis <jmdavisProg@gmx.com> 2011-05-05 01:26:50 PDT --- Fixed: https://github.com/D-Programming-Language/phobos/commit/a34673cc82337b3004d9e3526b43711ec66b3727 -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
Copyright © 1999-2021 by the D Language Foundation