Re: [Telepathy] rework of the UI

On Tue, 2006-09-12 at 08:13 +0100, Martyn Russell wrote:
> On Tue, 2006-09-12 at 00:43 +0200, Xavier Claessens wrote:
> > Here is what I have so far [1].
> > 
> > 1) code cleanup in many places
> > 
> > 2) removes those modules and merge changes in normal module:
> > gossip-new-account-window-telepathy.[ch]
> > gossip-accounts-dialog-telepathy.[ch]
> > 
> > 3) As said GossipAccount uses the GHashTable for
> > gossip_account_get/set_*(). Rework the whole API.
> 
> The gossip_account_() API should probably use g_dataset_() here instead
> of g_hash_table_(). Not sure though.

Do you mean, g_datalist? if so I agree. I'm not sure what the benefits
of g_dataset_() are.

> 
> > 4) Removes GossipAccountParamFlags to simplify the code. It's useless
> > for now but should be added back in some way later I think.
> > 
> > --> The DTD doesn't validate for account xml files, I don't understand
> > why, is there a guru of xml here to fix that ?
> 
> Yea, there is a libgossip/gossip-account.dtd which we validate the
> ~/.gnome2/Gossip/accounts.xml with. It requires a bunch of values, like
> id, password, etc which are not necessary for ALL protocols. This can
> change to just require the essentials, like an account name and
> auto_connect feature. Whatever is consistent across protocols. 
> 
> _______________________________________________
> Gossip-dev mailing list
> [email protected]
> http://lists.imendio.com/mailman/listinfo/gossip-dev