[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] About KOrganizer's timezone setting
From:       Allen Winter <winter () kde ! org>
Date:       2007-07-31 21:11:30
Message-ID: 200707311711.31198.winter () kde ! org
[Download RAW message or body]

On Monday 30 July 2007 11:56:15 am 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)
> 
> Bruno
> 

Committed.

-Allen
_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]