aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--irc.h4
-rw-r--r--irc_channel.c49
-rw-r--r--protocols/account.c88
3 files changed, 39 insertions, 102 deletions
diff --git a/irc.h b/irc.h
index 42d3a721..c96cdad3 100644
--- a/irc.h
+++ b/irc.h
@@ -228,8 +228,6 @@ typedef struct irc_channel_user {
int flags;
} irc_channel_user_t;
-struct account;
-
typedef enum {
IRC_CC_TYPE_DEFAULT = 0x00001,
IRC_CC_TYPE_REST = 0x00002, /* Still not implemented. */
@@ -243,7 +241,7 @@ typedef enum {
struct irc_control_channel {
irc_control_channel_type_t type;
struct bee_group *group;
- GSList *account;
+ struct account *account;
struct prpl *protocol;
char modes[5];
};
diff --git a/irc_channel.c b/irc_channel.c
index 739360de..583b6d20 100644
--- a/irc_channel.c
+++ b/irc_channel.c
@@ -871,9 +871,9 @@ static char *set_eval_by_account(set_t *set, char *value)
struct irc_channel *ic = set->data;
struct irc_control_channel *icc = ic->data;
account_t *acc;
- GSList *accl, *new_acc = NULL;
- char **accounts, **account;
+ struct acclist *accl, *tmp, *new_acc = NULL;
+ char **accounts, **account;
accounts = g_strsplit(value, ",", 0);
if (accounts == NULL) {
goto fail;
@@ -883,14 +883,25 @@ static char *set_eval_by_account(set_t *set, char *value)
if (!(acc = account_get(ic->irc->b, *account))) {
goto fail;
} else {
- new_acc = g_slist_append(new_acc, account);
+ tmp = g_malloc(sizeof(struct account));
+ if (tmp == NULL) {
+ goto fail;
+ } else {
+ tmp->acc = acc;
+ tmp->next = new_acc;
+ new_acc = tmp;
+ }
}
}
- accl = icc->account;
- g_slist_free(accl);
+ accl = (struct acclist *)icc->account;
+ while (accl) {
+ tmp = accl->next;
+ g_free(accl);
+ accl = tmp;
+ }
- icc->account = new_acc;
+ icc->account = (struct account *)new_acc;
if ((icc->type & IRC_CC_TYPE_MASK) == IRC_CC_TYPE_ACCOUNT) {
bee_irc_channel_update(ic->irc, ic, NULL);
}
@@ -900,7 +911,13 @@ static char *set_eval_by_account(set_t *set, char *value)
return g_strdup(value);
fail:
- g_slist_free(new_acc);
+ accl = new_acc;
+ while (accl) {
+ tmp = accl->next;
+ g_free(accl);
+ accl = tmp;
+ }
+
g_strfreev(accounts);
return SET_INVALID;
@@ -1027,7 +1044,7 @@ fail:
gboolean irc_channel_wants_user(irc_channel_t *ic, irc_user_t *iu)
{
struct irc_control_channel *icc = ic->data;
- GSList *accl;
+ struct acclist *accl;
gboolean ret = FALSE;
if (iu->bu == NULL) {
@@ -1039,12 +1056,13 @@ gboolean irc_channel_wants_user(irc_channel_t *ic, irc_user_t *iu)
ret = iu->bu->group == icc->group;
break;
case IRC_CC_TYPE_ACCOUNT:
- for (accl = icc->account; accl; accl = accl->next) {
- account_t *acc = accl->data;
- if (iu->bu->ic->acc == acc) {
+ accl = (struct acclist *)icc->account;
+ while (accl) {
+ if (iu->bu->ic->acc == accl->acc) {
ret = TRUE;
break;
}
+ accl = accl->next;
}
break;
case IRC_CC_TYPE_PROTOCOL:
@@ -1066,6 +1084,7 @@ gboolean irc_channel_wants_user(irc_channel_t *ic, irc_user_t *iu)
static gboolean control_channel_free(irc_channel_t *ic)
{
struct irc_control_channel *icc = ic->data;
+ struct acclist *accl, *tmp;
set_del(&ic->set, "account");
set_del(&ic->set, "fill_by");
@@ -1073,7 +1092,13 @@ static gboolean control_channel_free(irc_channel_t *ic)
set_del(&ic->set, "protocol");
set_del(&ic->set, "show_users");
- g_slist_free(icc->account);
+ accl = (struct acclist *)icc->account;
+ while (accl) {
+ tmp = accl->next;
+ g_free(accl);
+ accl = tmp;
+ }
+
g_free(icc);
ic->data = NULL;
diff --git a/protocols/account.c b/protocols/account.c
index 6f6ad339..e25e40c7 100644
--- a/protocols/account.c
+++ b/protocols/account.c
@@ -163,55 +163,14 @@ char *set_eval_account(set_t *set, char *value)
return NULL; /* password shouldn't be visible in plaintext! */
} else if (strcmp(set->key, "tag") == 0) {
account_t *oa;
- irc_t *irc;
- GSList *l;
- char *old;
/* Enforce uniqueness. */
if ((oa = account_by_tag(acc->bee, value)) && oa != acc) {
return SET_INVALID;
}
- old = acc->tag;
+ g_free(acc->tag);
acc->tag = g_strdup(value);
-
- irc = acc->bee->ui_data;
- for (l = irc->channels; l; l = l->next) {
- irc_channel_t *ic = l->data;
-
- if (g_strcasecmp(set_getstr(&ic->set, "type"), "control") == 0) {
- gboolean found = FALSE;
- char **account, **accounts;
- char *saccount = set_getstr(&ic->set, "account");
-
- if (saccount == NULL || *saccount == '\0') {
- continue;
- }
-
- accounts = g_strsplit(saccount, ",", 0);
- for (account = accounts; *account; account++) {
- if (g_strcasecmp(*account, old) == 0) {
- fprintf(stderr, "Found\n");
- found = TRUE;
- g_free(*account);
- *account = g_strdup(value);
- break;
- }
- }
-
- if (found) {
- saccount = g_strjoinv(",", accounts);
- g_strfreev(accounts);
-
- fprintf(stderr, "%s\n", saccount);
- set_setstr(&ic->set, "account", saccount);
-
- g_free(saccount);
- }
- }
- }
-
- g_free(old);
return value;
} else if (strcmp(set->key, "auto_connect") == 0) {
if (!is_bool(value)) {
@@ -335,8 +294,6 @@ account_t *account_by_tag(bee_t *bee, const char *tag)
void account_del(bee_t *bee, account_t *acc)
{
account_t *a, *l = NULL;
- GSList *accl;
- irc_t *irc;
if (acc->ic) {
/* Caller should have checked, accounts still in use can't be deleted. */
@@ -360,49 +317,6 @@ void account_del(bee_t *bee, account_t *acc)
}
*/
- irc = acc->bee->ui_data;
- for (accl = irc->channels; accl; accl = accl->next) {
- irc_channel_t *ic = accl->data;
-
- if (g_strcasecmp(set_getstr(&ic->set, "type"), "control") == 0) {
- gboolean found = FALSE;
- char **account, **accounts;
- char *saccount = set_getstr(&ic->set, "account");
-
- if (saccount == NULL || *saccount == '\0') {
- continue;
- }
-
- accounts = g_strsplit(saccount, ",", 0);
- for (account = accounts; *account; account++) {
- if (g_strcasecmp(*account, acc->tag) == 0) {
- found = TRUE;
- break;
- }
- }
-
- if (found) {
- g_free(*account);
-
- for (;;) {
- *account = *(account + 1);
-
- account++;
- if (*account == NULL) {
- break;
- }
- }
-
- saccount = g_strjoinv(",", accounts);
- g_strfreev(accounts);
-
- set_setstr(&ic->set, "account", saccount);
-
- g_free(saccount);
- }
- }
- }
-
while (a->set) {
set_del(&a->set, a->set->key);
}