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

List:       kopete-devel
Subject:    Re: [Kopete-devel] [PATCH] KopeteContact property system + ICQ
From:       Stefan Gehn <sgehn () gmx ! net>
Date:       2004-01-24 19:49:40
Message-ID: 200401242049.40290.sgehn () gmx ! net
[Download RAW message or body]

On Samstag Januar 24 2004 20:24, Martijn Klingens wrote:
> On Saturday 24 January 2004 20:10, Stefan Gehn wrote:
> > I doubt it's half the lines it was before :)
>
> You're close though, especially considering that it's a much smaller patch
> for me to read now, only 10.7kb libkopete as opposed to the 28kb monster
> you sent yesterday ;)
>
> I haven't really checked the Oscar diff, that's not my territory, but as
> for libkopete, feel free to commit.

K, will do.

> One gripe against the Oscar part that I just spotted: please consider using
> the address book fields for things like email addresses instead of the
> properties framework. Basically you should use this new API for things that
> do NOT belong in the address book, and I hope that it's obvious that email
> address are not amongst those things :)

And how does it work? All I have found is KopetePlugin::addAddressBookField() 
and I fail to see what this has to do with saving a ton of user-information 
somewhere in the xml.

Bye, Stefan aka mETz
-- 
ICQ#51123152 | Moege der Pinguin mit euch sein
_______________________________________________
Kopete-devel mailing list
Kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

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