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

List:       kde-pim
Subject:    Re: [Kde-pim] korganizer patch
From:       Cornelius Schumacher <schumacher () kde ! org>
Date:       2002-11-28 16:37:07
[Download RAW message or body]

On Thursday 28 November 2002 16:45, Bo Thorsen wrote:
>
> This is the first patch I have come up with. It solves a problem that is
> introduced in kroupware branch because there is stuff we need to set up
> that is also set up in koapp. The solution was to move this to a file in
> it's own. I have already checked this solution into kroupware_branch and
> made the necessary change to kokroupware.cpp.

This looks good, but I would like to slightly change the semantic of the new 
class. I was planning to put all the code for communication with the alarm 
daemon in an own class anyway, so I would like to rename it from KOAppShared 
to AlarmClient. Would that be ok?

-- 
Cornelius Schumacher <schumacher@kde.org>

_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
http://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]