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

List:       kde-pim
Subject:    Re: [Kde-pim] libkabc API problems
From:       Tobias Koenig <tokoe82 () yahoo ! de>
Date:       2002-09-29 20:43:59
[Download RAW message or body]

On Sun, Sep 29, 2002 at 03:23:49AM +0200, Cornelius Schumacher wrote:
Hi Cornelius,

> There are some small API problems in libkabc which come from changes 
> after the 3.0 relelase. As we can't fix them after we have released the 
> lib because of binary compatibility we have to resolve this now:
> 
> - AddressBook::saveAll() undermines the locking concept. I would like to 
> remove this function. It's the responsibility of the application to 
> ensure that all resources are saved.
I've moved the code from AddressBook::saveAll() to StdAddressBook::save()
now.

> - AddressBook::resourceAddress() is in the wrong class. It doesn't 
> operate on the AddressBook but on the Resource, so I think it should be 
> part of Resource. I propose renaming it to Resource::takeAddressee().
This method is removed completely now, since it fault by design (will write
you a private mail concerning it).

> - AddressBook::distributionListChanged() does not belong to AddressBook. 
> Distribution lists work on top of AddressBook. The AddressBook should 
> not depend on any distribution list classes. This code should probably 
> moved to the distribution list classes.
No, AddressBook watches for changes in $KDEDIR/share/apps/kabc/distlist
and emits addressbookChanged() if so. So applications, which displays a
distribution list selection can update their informations.

> - Where is AddressBook::lastAddressee() used? 
Removed it now.

Ciao,
Tobias
-- 
In a world without walls and fences who
needs Windows and Gates???
_______________________________________________
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]