Mercurial > pidgin
view src/protocols/zephyr/init_et.c @ 2347:182a4f8b1333
[gaim-migrate @ 2360]
Sorry rob. I just don't think that you should be committing what you know is half-complete code, and allow users to access it. We can't be recommending to people to be using CVS instead of releases and then add things that we know beforehand are buggy or incomplete. Also I don't think that you've implemented it properly: More than just Oscar have buddy icons (though Jabber is reworking their buddy icon stuff) and so it shouldn't be protocol-specific like this; also it should apply to the aim_user and not the gaim_connection, and you should be able to set it while you're not online, which means it should go in the Modify Account dialog and not the Protocol Actions dialog, with an OPT_PROTO_BUDDY_ICON flag so that the dialog knows whether or not the buddy icon can be set.
Also, you broke -Wall.
committer: Tailor Script <tailor@pidgin.im>
author | Eric Warmenhoven <eric@warmenhoven.org> |
---|---|
date | Sat, 22 Sep 2001 10:08:19 +0000 |
parents | 424a40f12a6c |
children | 43dec50d8d51 |
line wrap: on
line source
/* * $Header$ * $Source$ * $Locker$ * * Copyright 1986, 1987, 1988 by MIT Information Systems and * the MIT Student Information Processing Board. * * For copyright info, see mit-sipb-copyright.h. */ #include <sysdep.h> #include "error_table.h" #include "mit-sipb-copyright.h" #ifndef lint static const char rcsid_init_et_c[] = "$Header$"; #endif struct foobar { struct et_list etl; struct error_table et; }; extern struct et_list * _et_list; int init_error_table(msgs, base, count) const char * const * msgs; int base; int count; { struct foobar * new_et; if (!base || !count || !msgs) return 0; new_et = (struct foobar *) malloc(sizeof(struct foobar)); if (!new_et) return errno; /* oops */ new_et->etl.table = &new_et->et; new_et->et.msgs = msgs; new_et->et.base = base; new_et->et.n_msgs= count; new_et->etl.next = _et_list; _et_list = &new_et->etl; return 0; }