[Kde-pim] About KOrganizer's timezone setting
David Jarvie
lists at astrojar.org.uk
Mon Jul 30 17:55:11 BST 2007
On Monday 30 July 2007 16:56, Bruno Virlet wrote:
> On Monday 30 July 2007 17:11:35 David Jarvie wrote:
>> Your patch looks fine except that the API comments don't tally with the
>> implementation code. The "spec" parameter has the following description:
>>
>> "If set, return the date in the given spec, else use the system spec"
>>
>> In fact the patch code doesn't use the system spec, but instead uses
>> whatever spec the incidence uses (which could of course be different for
>> different incidences). If by system spec you mean the computer's system
>> time zone, you need to use KSystemTimeZones::local().
> Attached is the correct patch (will also takes todo.cpp/h in account)
That looks fine. :)
--
David Jarvie.
KAlarm author & maintainer.
http://www.astrojar.org.uk/kalarm
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list