Re: [Telepathy] rework of the UI

On Wed, 2006-09-13 at 17:13 +0200, Richard Hult wrote:
> Hi,
> 
> I just took a very quick look at the patch:
> 
> * Please try to avoid whitespace-only changes
> 
> * Use g_object_set() instead of setting up a gvalue and using
> g_object_set_property (and when using g_object_set, don't cast the object).
> 
> * g_value_transform looks suspicious, is that right? don't you just want
> to copy the value?
> 
> * What's the reason for having the function that returns a hash table
> with all params?

I just had an idea: instead of returning a hashtable, we could have a
function called gossip_account_params_foreach() - One more step in
hiding the implementation without having to iterate over all the
parameters a million times.

> 
> * The quark args should not be called param_name, since they are not
> names, but quarks, however...
> 
> * ...what's the reason really to have a datalist, when a hashtable would
> work just as fine and probably be simpler and less code?
> 
> /Richard
>