Bug 5144 – Issue with SYSTEMTIME2d_time daylightSavingTA()
Status
RESOLVED
Resolution
WONTFIX
Severity
normal
Priority
P2
Component
phobos
Product
D
Version
D1 (retired)
Platform
Other
OS
Windows
Creation time
2010-10-31T06:26:46Z
Last change time
2017-08-17T15:20:58Z
Assigned to
No Owner
Creator
Robert
Comments
Comment #0 by koine4895 — 2010-10-31T06:26:46Z
The last two parameters to dateFromNthWeekdayOfMonth should be swapped in SYSTEMTIME2d_time:
auto mday = dateFromNthWeekdayOfMonth(year,
st.wMonth, st.wDay, st.wDayOfWeek);
should be:
auto mday = dateFromNthWeekdayOfMonth(year,
st.wMonth, st.wDayOfWeek, st.wDay);
Also, daylightSavingTA() is calling the Windows API GetTimeZoneInformation(), which returns the time in LOCAL time, but I believe it is assuming that it is UTC time.
Comment #1 by bugzilla — 2011-02-03T02:51:44Z
Marking this as D1-only, since std.datetime has replaced std.date in D1.
Comment #2 by bugzilla — 2011-02-03T02:53:57Z
(In reply to comment #1)
> ...std.datetime has replaced std.date in D1.
And by D1 I do of course mean D2.
Comment #3 by razvan.nitu1305 — 2017-08-17T14:21:43Z
Is D1 maintained anymore? Shouldn't we close this?
Comment #4 by schveiguy — 2017-08-17T15:20:58Z
(In reply to RazvanN from comment #3)
> Is D1 maintained anymore? Shouldn't we close this?
Some industry leaders (e.g. sociomantic) use D1, but I believe nobody uses D1 Phobos.