From 97d1b964c3b998116f05fd41f26bfe2f724e3150 Mon Sep 17 00:00:00 2001 From: James Booth Date: Mon, 23 Nov 2015 23:54:14 +0000 Subject: [PATCH] Created config/conflists.c for shared config list handling --- Makefile.am | 2 ++ src/config/accounts.c | 58 ++++-------------------------- src/config/conflists.c | 82 ++++++++++++++++++++++++++++++++++++++++++ src/config/conflists.h | 38 ++++++++++++++++++++ 4 files changed, 129 insertions(+), 51 deletions(-) create mode 100644 src/config/conflists.c create mode 100644 src/config/conflists.h diff --git a/Makefile.am b/Makefile.am index 21b0d3aa..1f7e0350 100644 --- a/Makefile.am +++ b/Makefile.am @@ -35,6 +35,7 @@ core_sources = \ src/tools/p_sha1.h src/tools/p_sha1.c \ src/tools/autocomplete.c src/tools/autocomplete.h \ src/tools/tinyurl.c src/tools/tinyurl.h \ + src/config/conflists.c src/config/conflists.h \ src/config/accounts.c src/config/accounts.h \ src/config/tlscerts.c src/config/tlscerts.h \ src/config/account.c src/config/account.h \ @@ -67,6 +68,7 @@ unittest_sources = \ src/config/preferences.c src/config/preferences.h \ src/config/theme.c src/config/theme.h \ src/config/scripts.c src/config/scripts.h \ + src/config/conflists.c src/config/conflists.h \ src/window_list.c src/window_list.h \ src/event/server_events.c src/event/server_events.h \ src/event/client_events.c src/event/client_events.h \ diff --git a/src/config/accounts.c b/src/config/accounts.c index 2d4581ee..234bed1f 100644 --- a/src/config/accounts.c +++ b/src/config/accounts.c @@ -42,6 +42,7 @@ #include "common.h" #include "config/account.h" +#include "config/conflists.h" #include "jid.h" #include "log.h" #include "tools/autocomplete.h" @@ -55,7 +56,6 @@ static Autocomplete enabled_ac; static void _save_accounts(void); static gchar* _get_accounts_file(void); -static void _remove_from_list(GKeyFile *accounts, const char *const account_name, const char *const key, const char *const contact_jid); void accounts_load(void) @@ -604,66 +604,22 @@ accounts_add_otr_policy(const char *const account_name, const char *const contac // check for and remove from other lists if (strcmp(policy, "manual") == 0) { - _remove_from_list(accounts, account_name, "otr.opportunistic", contact_jid); - _remove_from_list(accounts, account_name, "otr.always", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.opportunistic", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.always", contact_jid); } if (strcmp(policy, "opportunistic") == 0) { - _remove_from_list(accounts, account_name, "otr.manual", contact_jid); - _remove_from_list(accounts, account_name, "otr.always", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.manual", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.always", contact_jid); } if (strcmp(policy, "always") == 0) { - _remove_from_list(accounts, account_name, "otr.opportunistic", contact_jid); - _remove_from_list(accounts, account_name, "otr.manual", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.opportunistic", contact_jid); + conf_string_list_remove(accounts, account_name, "otr.manual", contact_jid); } _save_accounts(); } } -static void -_remove_from_list(GKeyFile *accounts, const char *const account_name, const char *const key, const char *const contact_jid) -{ - gsize length; - gchar **list = g_key_file_get_string_list(accounts, account_name, key, &length, NULL); - - if (list) { - int i = 0; - GList *glist = NULL; - gboolean deleted = FALSE; - - for (i = 0; i < length; i++) { - // item found, mark as deleted - if (strcmp(list[i], contact_jid) == 0) { - deleted = TRUE; - } else { - // add item to our g_list - glist = g_list_append(glist, strdup(list[i])); - } - } - - if (deleted) { - if (g_list_length(glist) == 0) { - g_key_file_remove_key(accounts, account_name, key, NULL); - } else { - // create the new list entry - const gchar* new_list[g_list_length(glist)+1]; - GList *curr = glist; - i = 0; - while (curr) { - new_list[i++] = strdup(curr->data); - curr = g_list_next(curr); - } - new_list[i] = NULL; - g_key_file_set_string_list(accounts, account_name, key, new_list, g_list_length(glist)); - } - } - - g_list_free_full(glist, g_free); - } - - g_strfreev(list); -} - void accounts_set_muc_service(const char *const account_name, const char *const value) { diff --git a/src/config/conflists.c b/src/config/conflists.c new file mode 100644 index 00000000..bf5bc482 --- /dev/null +++ b/src/config/conflists.c @@ -0,0 +1,82 @@ +/* + * conflists.c + * + * Copyright (C) 2012 - 2015 James Booth + * + * This file is part of Profanity. + * + * Profanity is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Profanity is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with Profanity. If not, see . + * + * In addition, as a special exception, the copyright holders give permission to + * link the code of portions of this program with the OpenSSL library under + * certain conditions as described in each individual source file, and + * distribute linked combinations including the two. + * + * You must obey the GNU General Public License in all respects for all of the + * code used other than OpenSSL. If you modify file(s) with this exception, you + * may extend this exception to your version of the file(s), but you are not + * obligated to do so. If you do not wish to do so, delete this exception + * statement from your version. If you delete this exception statement from all + * source files in the program, then also delete it here. + * + */ + +#include +#include + +gboolean +conf_string_list_remove(GKeyFile *keyfile, const char *const group, const char *const key, const char *const item) +{ + gsize length; + gchar **list = g_key_file_get_string_list(keyfile, group, key, &length, NULL); + + gboolean deleted = FALSE; + if (list) { + int i = 0; + GList *glist = NULL; + + for (i = 0; i < length; i++) { + // item found, mark as deleted + if (strcmp(list[i], item) == 0) { + deleted = TRUE; + } else { + // add item to our g_list + glist = g_list_append(glist, strdup(list[i])); + } + } + + if (deleted) { + if (g_list_length(glist) == 0) { + g_key_file_remove_key(keyfile, group, key, NULL); + } else { + // create the new list entry + const gchar* new_list[g_list_length(glist)+1]; + GList *curr = glist; + i = 0; + while (curr) { + new_list[i++] = strdup(curr->data); + curr = g_list_next(curr); + } + new_list[i] = NULL; + g_key_file_set_string_list(keyfile, group, key, new_list, g_list_length(glist)); + } + } + + g_list_free_full(glist, g_free); + } + + g_strfreev(list); + + return deleted; +} diff --git a/src/config/conflists.h b/src/config/conflists.h new file mode 100644 index 00000000..f67e58b8 --- /dev/null +++ b/src/config/conflists.h @@ -0,0 +1,38 @@ +/* + * conflists.h + * + * Copyright (C) 2012 - 2015 James Booth + * + * This file is part of Profanity. + * + * Profanity is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Profanity is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with Profanity. If not, see . + * + * In addition, as a special exception, the copyright holders give permission to + * link the code of portions of this program with the OpenSSL library under + * certain conditions as described in each individual source file, and + * distribute linked combinations including the two. + * + * You must obey the GNU General Public License in all respects for all of the + * code used other than OpenSSL. If you modify file(s) with this exception, you + * may extend this exception to your version of the file(s), but you are not + * obligated to do so. If you do not wish to do so, delete this exception + * statement from your version. If you delete this exception statement from all + * source files in the program, then also delete it here. + * + */ + +#include + +gboolean conf_string_list_remove(GKeyFile *keyfile, const char *const group, const char *const key, + const char *const item);