Re: final account rework

Xavier Claessens <[email protected]> Fri, 06 Oct 2006 23:18:32 +0200
Newsgroups gmane.comp.gnome.gossip.devel
Message-ID <1160169512.19956.4.camel@zdra-desktop>
Le vendredi 06 octobre 2006 à 22:45 +0200, Richard Hult a écrit :
> Xavier Claessens skrev:
> 
> > The patch
> > http://users.skynet.be/Zdra/gossip-account.patch
> 
> This patch looks OK to me in general. I assume that you have tested it
> so that it works and so on :)

It works for me. I would be happy if others can test it too :-)

> I would call the new_account function create_account.

Ok.

> > New files in src/
> > http://users.skynet.be/Zdra/gossip-account-jabber.c
> > http://users.skynet.be/Zdra/gossip-account-jabber.h
> > http://users.skynet.be/Zdra/gossip-account-unknown.c
> > http://users.skynet.be/Zdra/gossip-account-unknown.h
> 
> Should the latter be called -generic? Unknown sounds a bit odd to me.
> I'd also prefer to have some kind of indication that we are talking
> about a dialog/window here, gossip-account-dialog-jabber for example.

Ok I can rename to gossip-account-generic. It's not a window nor a
dialog, it's a widget... So what about gossip-account-jabber-widget ?

Xavier.

_______________________________________________
Gossip-dev mailing list
[email protected]
http://lists.imendio.com/mailman/listinfo/gossip-dev
signature.asc (application/pgp-signature, 189 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)

iD8DBQBFJsgo6dEBUn2qPAMRAsFxAJwI+HaXarBBQ8LXZiqTfw2kiGDXWgCcDpKm
XxveugM2P/H/Cu0hoPkBDQI=
=1SqN
-----END PGP SIGNATURE-----