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

List:       kde-pim
Subject:    Re: [Kde-pim] PATCH: addIncidence() DCOP call for KOrganizer
From:       Allen Winter <winter () kde ! org>
Date:       2005-06-27 0:22:39
Message-ID: 200506262022.39670.winter () kde ! org
[Download RAW message or body]

On Sunday 26 June 2005 08:14 pm, David Jarvie wrote:
> Here is a patch to add a new DCOP call to KOrganizer. The new method in 
> korganizeriface.h,  addIncidence(QString& ical), allows another application 
> to send KOrganizer an iCalendar incidence in string format, which KOrganizer 
> adds to its default calendar without showing any dialog.
> 
> The purpose is to enable KAlarm to add events to KOrganizer, as per bug 
> wishlist number 90315. I have written it in a general form to allow any 
> incidence to be added.
> 
> Please review.
> 
> Cheers,
> David Jarvie.
> KAlarm author.
> 

David,
I'd like to see you check the return value of ICalFormat.fromString() and do the \
appropriate thing upon failure.

-- 
Let's Keep the Political Talk Out of KDE PLEASE
_______________________________________________
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]