[libical] icaltime_compare_date_only semantics

Dan Mosedale dmose at meer.net
Thu Aug 25 10:51:56 PDT 2005


I wrote:
> Right now, it appears to me that icaltime_compare_date_only() can't 
> possibly do the right thing in all circumstances, because it always 
> normalizes to UTC and then compares.  If I have two icaltime's that 
> are in the same day in some timezone, but that difference spans 
> midnight when normalized to UTC, the wrong thing is returned.
>
> I propose to fix this by giving icaltime_compare_date_only() a third 
> argument: timezone to do the compare in.  Both dates would be 
> normalized to that timezone before comparing. What do folks think -- 
> does this sound like a reasonable API change?
I've attached a very small patch which makes exactly this change; is 
this the proper way to get it considered for acceptance into the tree?

Dan

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: date-only-compare-libical-only.diff
URL: <http://lists.infradead.org/pipermail/libical-interest/attachments/20050825/e7dd8b94/attachment.ksh>


More information about the libical-interest mailing list