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

List:       kde-commits
Subject:    Re: branches/KDE/3.5/kdepim/libkcal
From:       Reinhold Kainhofer <reinhold () kainhofer ! com>
Date:       2005-07-29 14:56:16
Message-ID: 200507291656.21535.reinhold () kainhofer ! com
[Download RAW message or body]


Am Freitag, 29. Juli 2005 16:35 schrieb David Jarvie:
> SVN commit 440003 by djarvie:
>
> Add clear() method to clear all recurrence and exception rules & dates.

Thanks.

> Add setWeekly(freq, weekstart) and addWeeklyDays() to allow weekly
> recurrences to be set up in the same manner as other recurrence periods.

Let's get rid of setWeekly( freq, days, weekstart ) altogether! 
Several places can be simplified, since then you can initially set the type of 
the recurrence period, and later on add the type-specific settings. So far, 
you had to cache the fact that weekly is wanted, and call setWeekly only 
after you have determined the actual days. 

There are some places where the logic can be simplified by this.

Cheers,
Reinhold

-- 
------------------------------------------------------------------
Reinhold Kainhofer, Vienna, Austria
email: reinhold@kainhofer.com, http://reinhold.kainhofer.com/
 * Financial and Actuarial Mathematics, TU Wien, http://www.fam.tuwien.ac.at
 * K Desktop Environment, http://www.kde.org/, KOrganizer / KPilot maintainer

[Attachment #3 (application/pgp-signature)]

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