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

List:       kde-pim
Subject:    Re: [Kde-pim] PATCH, RFC: Korganizer and timezones (bug 68345)
From:       Shaheed <srhaque () iee ! org>
Date:       2003-12-16 20:03:31
[Download RAW message or body]

Reinhold Kainhofer wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Am Montag, 15. Dezember 2003 21:34 schrieb Shaheed:
>> BTW: did you notice that I only added the calls to add timezone support
>> for Events and not other types of object? I did so because I was not sure
>> which other dates might be subject to timezones...but if you think it
>> appropriate, I could take a stab at adding a few more calls to
>> readTzidParameter() and send here for review?
> 
> AFAIU, rfc2445 allows TZID parameters for all times stored in the ics
> file. So we should really make sure to add reading time zone information
> to all places where times occur.

Yes, that was my understanding too. Rather than pepper libkcalimpl.cpp with
these calls in an error-prone manner, how about if I do as the "TBD" says,
and patch libical instead? I realise it is an imported library, and this
will be a pain when we upgrade it, but...

Comments? Suggestions? Shaheed
_______________________________________________
kde-pim mailing list
kde-pim@mail.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] 

Configure | About | News | Add a list | Sponsored by KoreLogic