summaryrefslogtreecommitdiff
path: root/Libraries/LibCore/DateTime.cpp
diff options
context:
space:
mode:
authorNico Weber <thakis@chromium.org>2020-08-30 09:39:16 -0400
committerAndreas Kling <kling@serenityos.org>2020-08-30 16:56:47 +0200
commit9ad5a261f7fe03aac6d2f6261eb74026145781ea (patch)
treeff9ba96794f470e07181ec76f78494e2ce5954bd /Libraries/LibCore/DateTime.cpp
parent57dd3b66c5db7fc57d4e1c55f1515901e02bf419 (diff)
downloadserenity-9ad5a261f7fe03aac6d2f6261eb74026145781ea.zip
LibCore: Let DateTime::create()/set_time() take summer time into account
DateTime::create() takes a date/time in local time, but it set tm_isdst to 0, which meant it was in local winter time always. Set tm_isdst to -1 so that times during summer time are treated in summer time, and times in winter time are treated as winter time (when appropriate). When the time is adjusted backward by one hour, the same time can be in winter time or summer time, so this isn't 100% reliable, but for most of the year it should work fine. Since LibJS uses DateTime, this means that the Date tuple ctor (which creates a timestamp from year/month/day/hours/etc in local time) and getTime() should now have consistent (and correct) output, which should fix #3327. In Serenity itself, dst handling (and timezones) are unimplemented and this doens't have any effect yet, but in Lagom this has an effect.
Diffstat (limited to 'Libraries/LibCore/DateTime.cpp')
-rw-r--r--Libraries/LibCore/DateTime.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/Libraries/LibCore/DateTime.cpp b/Libraries/LibCore/DateTime.cpp
index 64c48a9075..58f914a629 100644
--- a/Libraries/LibCore/DateTime.cpp
+++ b/Libraries/LibCore/DateTime.cpp
@@ -88,6 +88,7 @@ void DateTime::set_time(unsigned year, unsigned month, unsigned day, unsigned ho
tm.tm_mday = (int)day;
tm.tm_mon = (int)month - 1;
tm.tm_year = (int)year - 1900;
+ tm.tm_isdst = -1;
// mktime() doesn't read tm.tm_wday and tm.tm_yday, no need to fill them in.
m_timestamp = mktime(&tm);