Review Request: Fix KSystemTimeZonesPrivate::setLocalZone()

Jon Severinsson jon at severinsson.net
Fri Oct 26 14:53:59 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107038/
-----------------------------------------------------------

(Updated Oct. 26, 2012, 2:53 p.m.)


Review request for KDE Frameworks.


Changes
-------

Fix regression found by kdatetimetest.


Description
-------

If the time zone file for the kconfig configured local time zone does not
exist in the kconfig configured zoneinfo directory, setLocalZone() would
set a time zone where isValid() returns true, but all time zone information
is garbage. This commit changes setLocalZone() to instead set the utc fallback
time zone as the local time zone.


Diffs (updated)
-----

  kdecore/date/ksystemtimezone.cpp aaa10fd 
  kdecore/tests/ktimezonestest.h 16ba1d0 
  kdecore/tests/ktimezonestest.cpp 8ea48a4 

Diff: http://git.reviewboard.kde.org/r/107038/diff/


Testing
-------


Thanks,

Jon Severinsson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20121026/fcad0e43/attachment.html>


More information about the Kde-frameworks-devel mailing list