[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