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

List:       kde-pim
Subject:    Re: [Kde-pim] [PATCH] Calendar class: libkcal, kresources, Kalarm,
From:       Reinhold Kainhofer <reinhold () kainhofer ! com>
Date:       2005-03-11 10:44:02
Message-ID: 200503111144.04662.reinhold () kainhofer ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 09 March 2005 20:08, Allen Winter wrote:
> - big cleanup and doxygenation
>
> Please review and let me know what you think.

Thanks a lot for the patches. I quickly looked at the libkcal and korganizer 
patches, and I didn't see any obvious problem. 

I'm fine with applying these. We've talked about the change from events(const 
QDate &, bool sorted) to finer-grained sort-directives already some time ago, 
so I don't think anyone should have a problem with these changes... After 
all, most applications don't use the sorting currently.

Reinhold

-- 
------------------------------------------------------------------
Reinhold Kainhofer, Vienna University of Technology, 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 #5 (application/pgp-signature)]

_______________________________________________
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] 

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