[Gossip-dev] final account rework

Xavier Claessens xclaesse at gmail.com
Sun Oct 1 18:46:16 CEST 2006


Hi,

I'm working on protocol specific UI for creating and editing accounts
[1]. This work is made for HEAD and will be very useful for telepathy.

One of the goal of this is removing the deprecated
gossip_account_[get,set]_*() API and use instead the new API
gossip_account_param_set(). I'm surprised there is not many places where
the deprecated API is used, most of it is in my new
gossip-account-jabber.[ch] module, that's why I'm thinking about
removing this API completely now.

For that I need one think: the protocol should create new accounts with
default values for all parameters. Something like
	GossipAccount *gossip_protocol_create_account (GossipProtocol*);
With telepathy it is already the protocol that creates new accounts and
it already sets all parameters with default values (Eitan correct me if
I miss understood the code). In telepathy we directly call
	gossip_telepathy_cmgr_new_account_from_protocol
from protocol/telepathy/ which violate the libgossip abstraction.

Is that a good idea if I implement all that in my next big account
rework ? Maybe it's better to make many little patches than one big ?

btw, new_account_window_get_account_info() has changed in last commit in
the THELEPATHY branch and don't have the telepathy specific code
anymore ! what happened ?

Xavier Claessens.

[1] http://bugzilla.gnome.org/show_bug.cgi?id=358099
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Ceci est une partie de message
	=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=
Url : http://lists.imendio.com/pipermail/gossip-dev/attachments/20061001/3e3ec9c3/attachment.pgp


More information about the Gossip-dev mailing list