From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp1 ([2001:41d0:2:c151::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms11 with LMTPS id SAVfCILaQGB1bgAA0tVLHw (envelope-from ) for ; Thu, 04 Mar 2021 13:02:58 +0000 Received: from aspmx2.migadu.com ([2001:41d0:2:c151::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp1 with LMTPS id iLgLBILaQGCNbgAAbx9fmQ (envelope-from ) for ; Thu, 04 Mar 2021 13:02:58 +0000 Received: from mail.notmuchmail.org (nmbug.tethera.net [IPv6:2607:5300:201:3100::1657]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by aspmx2.migadu.com (Postfix) with ESMTPS id 86C3B25775 for ; Thu, 4 Mar 2021 14:02:57 +0100 (CET) Received: from nmbug.tethera.net (localhost [127.0.0.1]) by mail.notmuchmail.org (Postfix) with ESMTP id 56E6527571; Thu, 4 Mar 2021 08:02:09 -0500 (EST) Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197]) by mail.notmuchmail.org (Postfix) with ESMTP id 2969A26ACD for ; Thu, 4 Mar 2021 08:01:54 -0500 (EST) Received: by fethera.tethera.net (Postfix, from userid 1001) id 1F6C95FA96; Thu, 4 Mar 2021 08:01:54 -0500 (EST) Received: (nullmailer pid 4047218 invoked by uid 1000); Thu, 04 Mar 2021 13:01:45 -0000 From: David Bremner To: notmuch@notmuchmail.org Cc: David Bremner Subject: [PATCH 27/27] CLI: rename notmuch_config_t to notmuch_conffile_t Date: Thu, 4 Mar 2021 09:01:39 -0400 Message-Id: <20210304130139.4046183-28-david@tethera.net> X-Mailer: git-send-email 2.30.1 In-Reply-To: <20210304130139.4046183-1-david@tethera.net> References: <20210304130139.4046183-1-david@tethera.net> MIME-Version: 1.0 Message-ID-Hash: W5UITGXRPQON3CDJVBZV3HJEHG6L2SAU X-Message-ID-Hash: W5UITGXRPQON3CDJVBZV3HJEHG6L2SAU X-MailFrom: bremner@tethera.net X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-notmuch.notmuchmail.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; suspicious-header X-Mailman-Version: 3.2.1 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Help: List-Post: List-Subscribe: List-Unsubscribe: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Migadu-Flow: FLOW_IN ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1614862977; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=9dRZGztIq4PUeJcmnPPwNISZjEcYB6ubEakWDa4lE98=; b=j3tOZPDMMZi4BNfcsdWiADe0gaQM962R2YvTdCQWKAJxn6qXbxnZYrEa1u9/FEuEMiUap0 ifaDH7ZxMzjGM1CBR5fWrsddvm07fKtRx1q1Jyb4pAl+AEEu0dVc0SotxAEycDNVNFG0kT jWCFhFkUanwd9+KXw1CV03f0rEs2mYWPEuV+CLKH8/kFM2GtlVuI63Sxnbr4dUoZYsgbx/ g2ZOGFIrfV7xD8IiNrDnhrDIO7yTvHoXUzNdrO8miS7tgPXbm5E6J1iM5q9JJUX5qBGAKl S0Tb3GYfljJ3v4eGnz/zpI1O8Ua1dsSxlHceVsdIhsjCX7Xz8GID3iSjd28vZw== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1614862977; a=rsa-sha256; cv=none; b=RDJv2iMmPsQuLy1/QxzX0m8VR9kebvNOOVXN7DWFlrgS7C32ggybZqqMF7ZC5ce6yJp6QK SoU/c1SaOL5v6WnB+u5M+d+GPiSudw8ofvXwST/TOfn1/M13b6b4n04i1jC4TRIcvQFREs /IhTVy/SuD+oEaqO/GCrruLOmvzg/iN6/ko6IXQlLpCFCZaBKCwlJJPhoqYZ/yUmKLViqU StXz0Kah40so9gvT9cHG3tm6gSnFkaVFooeuzw3JGKaCrEDUzpvTS1/nUAlLyQUcK12lSO nSPT8Hf8MQnw09l6fs/KYj85DJOUMmLfwmgEjnp8DT6jSMM1eDMLWNb58TSgjQ== ARC-Authentication-Results: i=1; aspmx2.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx2.migadu.com: domain of notmuch-bounces@notmuchmail.org designates 2607:5300:201:3100::1657 as permitted sender) smtp.mailfrom=notmuch-bounces@notmuchmail.org X-Migadu-Spam-Score: -0.80 Authentication-Results: aspmx2.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx2.migadu.com: domain of notmuch-bounces@notmuchmail.org designates 2607:5300:201:3100::1657 as permitted sender) smtp.mailfrom=notmuch-bounces@notmuchmail.org X-Migadu-Queue-Id: 86C3B25775 X-Spam-Score: -0.80 X-Migadu-Scanner: scn0.migadu.com X-TUID: prHMybR93325 The name is a compromise between keeping function names from getting too long, and avoiding confusion with the newer notmuch_config library API. Try to make clear that the residual functionality is only about updating configuration files. --- notmuch-client.h | 56 +++++++++++++++++++++++----------------------- notmuch-config.c | 58 ++++++++++++++++++++++++------------------------ notmuch-setup.c | 38 +++++++++++++++---------------- 3 files changed, 76 insertions(+), 76 deletions(-) diff --git a/notmuch-client.h b/notmuch-client.h index 1976352f..f071da86 100644 --- a/notmuch-client.h +++ b/notmuch-client.h @@ -156,7 +156,7 @@ chomp_newline (char *str) */ extern int notmuch_format_version; -typedef struct _notmuch_config notmuch_config_t; +typedef struct _notmuch_conffile notmuch_conffile_t; /* Commands that support structured output should support the * following argument @@ -260,55 +260,55 @@ typedef enum { NOTMUCH_COMMAND_CONFIG_LOAD = 1 << 5, } notmuch_command_mode_t; -notmuch_config_t * -notmuch_config_open (notmuch_database_t *notmuch, - const char *filename, - bool create); +notmuch_conffile_t * +notmuch_conffile_open (notmuch_database_t *notmuch, + const char *filename, + bool create); void -notmuch_config_close (notmuch_config_t *config); +notmuch_conffile_close (notmuch_conffile_t *config); int -notmuch_config_save (notmuch_config_t *config); +notmuch_conffile_save (notmuch_conffile_t *config); bool -notmuch_config_is_new (notmuch_config_t *config); +notmuch_conffile_is_new (notmuch_conffile_t *config); void -notmuch_config_set_database_path (notmuch_config_t *config, - const char *database_path); +notmuch_conffile_set_database_path (notmuch_conffile_t *config, + const char *database_path); void -notmuch_config_set_user_name (notmuch_config_t *config, - const char *user_name); +notmuch_conffile_set_user_name (notmuch_conffile_t *config, + const char *user_name); void -notmuch_config_set_user_primary_email (notmuch_config_t *config, - const char *primary_email); +notmuch_conffile_set_user_primary_email (notmuch_conffile_t *config, + const char *primary_email); void -notmuch_config_set_user_other_email (notmuch_config_t *config, - const char *other_email[], - size_t length); +notmuch_conffile_set_user_other_email (notmuch_conffile_t *config, + const char *other_email[], + size_t length); void -notmuch_config_set_new_tags (notmuch_config_t *config, - const char *new_tags[], - size_t length); +notmuch_conffile_set_new_tags (notmuch_conffile_t *config, + const char *new_tags[], + size_t length); void -notmuch_config_set_new_ignore (notmuch_config_t *config, - const char *new_ignore[], - size_t length); +notmuch_conffile_set_new_ignore (notmuch_conffile_t *config, + const char *new_ignore[], + size_t length); void -notmuch_config_set_maildir_synchronize_flags (notmuch_config_t *config, - bool synchronize_flags); +notmuch_conffile_set_maildir_synchronize_flags (notmuch_conffile_t *config, + bool synchronize_flags); void -notmuch_config_set_search_exclude_tags (notmuch_config_t *config, - const char *list[], - size_t length); +notmuch_conffile_set_search_exclude_tags (notmuch_conffile_t *config, + const char *list[], + size_t length); int notmuch_run_hook (notmuch_database_t *notmuch, const char *hook); diff --git a/notmuch-config.c b/notmuch-config.c index 0fca2adc..399ffccc 100644 --- a/notmuch-config.c +++ b/notmuch-config.c @@ -114,14 +114,14 @@ struct config_group { }, }; -struct _notmuch_config { +struct _notmuch_conffile { char *filename; GKeyFile *key_file; bool is_new; }; static int -notmuch_config_destructor (notmuch_config_t *config) +notmuch_conffile_destructor (notmuch_conffile_t *config) { if (config->key_file) g_key_file_free (config->key_file); @@ -130,7 +130,7 @@ notmuch_config_destructor (notmuch_config_t *config) } static bool -get_config_from_file (notmuch_config_t *config, bool create_new) +get_config_from_file (notmuch_conffile_t *config, bool create_new) { #define BUF_SIZE 4096 char *config_str = NULL; @@ -245,21 +245,21 @@ get_config_from_file (notmuch_config_t *config, bool create_new) * The default configuration also contains comments to guide the * user in editing the file directly. */ -notmuch_config_t * -notmuch_config_open (notmuch_database_t *notmuch, - const char *filename, - bool create) +notmuch_conffile_t * +notmuch_conffile_open (notmuch_database_t *notmuch, + const char *filename, + bool create) { char *notmuch_config_env = NULL; - notmuch_config_t *config = talloc_zero (notmuch, notmuch_config_t); + notmuch_conffile_t *config = talloc_zero (notmuch, notmuch_conffile_t); if (config == NULL) { fprintf (stderr, "Out of memory.\n"); return NULL; } - talloc_set_destructor (config, notmuch_config_destructor); + talloc_set_destructor (config, notmuch_conffile_destructor); if (filename) { config->filename = talloc_strdup (config, filename); @@ -294,14 +294,14 @@ notmuch_config_open (notmuch_database_t *notmuch, return config; } -/* Close the given notmuch_config_t object, freeing all resources. +/* Close the given notmuch_conffile_t object, freeing all resources. * * Note: Any changes made to the configuration are *not* saved by this - * function. To save changes, call notmuch_config_save before - * notmuch_config_close. + * function. To save changes, call notmuch_conffile_save before + * notmuch_conffile_close. */ void -notmuch_config_close (notmuch_config_t *config) +notmuch_conffile_close (notmuch_conffile_t *config) { talloc_free (config); } @@ -314,7 +314,7 @@ notmuch_config_close (notmuch_config_t *config) * printing a description of the error to stderr). */ int -notmuch_config_save (notmuch_config_t *config) +notmuch_conffile_save (notmuch_conffile_t *config) { size_t length; char *data, *filename; @@ -364,20 +364,20 @@ notmuch_config_save (notmuch_config_t *config) } bool -notmuch_config_is_new (notmuch_config_t *config) +notmuch_conffile_is_new (notmuch_conffile_t *config) { return config->is_new; } static void -_config_set (notmuch_config_t *config, +_config_set (notmuch_conffile_t *config, const char *group, const char *key, const char *value) { g_key_file_set_string (config->key_file, group, key, value); } static void -_config_set_list (notmuch_config_t *config, +_config_set_list (notmuch_conffile_t *config, const char *group, const char *key, const char *list[], size_t length) @@ -386,28 +386,28 @@ _config_set_list (notmuch_config_t *config, } void -notmuch_config_set_database_path (notmuch_config_t *config, +notmuch_conffile_set_database_path (notmuch_conffile_t *config, const char *database_path) { _config_set (config, "database", "path", database_path); } void -notmuch_config_set_user_name (notmuch_config_t *config, +notmuch_conffile_set_user_name (notmuch_conffile_t *config, const char *user_name) { _config_set (config, "user", "name", user_name); } void -notmuch_config_set_user_primary_email (notmuch_config_t *config, +notmuch_conffile_set_user_primary_email (notmuch_conffile_t *config, const char *primary_email) { _config_set (config, "user", "primary_email", primary_email); } void -notmuch_config_set_user_other_email (notmuch_config_t *config, +notmuch_conffile_set_user_other_email (notmuch_conffile_t *config, const char *list[], size_t length) { @@ -415,7 +415,7 @@ notmuch_config_set_user_other_email (notmuch_config_t *config, } void -notmuch_config_set_new_tags (notmuch_config_t *config, +notmuch_conffile_set_new_tags (notmuch_conffile_t *config, const char *list[], size_t length) { @@ -423,7 +423,7 @@ notmuch_config_set_new_tags (notmuch_config_t *config, } void -notmuch_config_set_new_ignore (notmuch_config_t *config, +notmuch_conffile_set_new_ignore (notmuch_conffile_t *config, const char *list[], size_t length) { @@ -431,7 +431,7 @@ notmuch_config_set_new_ignore (notmuch_config_t *config, } void -notmuch_config_set_search_exclude_tags (notmuch_config_t *config, +notmuch_conffile_set_search_exclude_tags (notmuch_conffile_t *config, const char *list[], size_t length) { @@ -582,7 +582,7 @@ notmuch_config_command_set (notmuch_database_t *notmuch, { char *group, *key; config_key_info_t *key_info; - notmuch_config_t *config; + notmuch_conffile_t *config; bool update_database = false; int opt_index, ret; char *item; @@ -625,7 +625,7 @@ notmuch_config_command_set (notmuch_database_t *notmuch, if (_item_split (item, &group, &key)) return 1; - config = notmuch_config_open (notmuch, + config = notmuch_conffile_open (notmuch, notmuch_config_path (notmuch), false); if (! config) return 1; @@ -650,9 +650,9 @@ notmuch_config_command_set (notmuch_database_t *notmuch, break; } - ret = notmuch_config_save (config); + ret = notmuch_conffile_save (config); - notmuch_config_close (config); + notmuch_conffile_close (config); return ret; } @@ -734,7 +734,7 @@ notmuch_config_command (notmuch_database_t *notmuch, int argc, char *argv[]) } void -notmuch_config_set_maildir_synchronize_flags (notmuch_config_t *config, +notmuch_conffile_set_maildir_synchronize_flags (notmuch_conffile_t *config, bool synchronize_flags) { g_key_file_set_boolean (config->key_file, diff --git a/notmuch-setup.c b/notmuch-setup.c index 22769a5a..e0afb1c4 100644 --- a/notmuch-setup.c +++ b/notmuch-setup.c @@ -131,7 +131,7 @@ notmuch_setup_command (notmuch_database_t *notmuch, size_t response_size = 0; GPtrArray *other_emails; notmuch_config_values_t *new_tags, *search_exclude_tags, *emails; - notmuch_config_t *config; + notmuch_conffile_t *config; #define prompt(format, ...) \ do { \ @@ -151,22 +151,22 @@ notmuch_setup_command (notmuch_database_t *notmuch, fprintf (stderr, "Warning: ignoring --uuid=%s\n", notmuch_requested_db_uuid); - config = notmuch_config_open (notmuch, - notmuch_config_path (notmuch), true); + config = notmuch_conffile_open (notmuch, + notmuch_config_path (notmuch), true); if (! config) return EXIT_FAILURE; - if (notmuch_config_is_new (config)) + if (notmuch_conffile_is_new (config)) welcome_message_pre_setup (); prompt ("Your full name [%s]: ", notmuch_config_get (notmuch, NOTMUCH_CONFIG_USER_NAME)); if (strlen (response)) - notmuch_config_set_user_name (config, response); + notmuch_conffile_set_user_name (config, response); prompt ("Your primary email address [%s]: ", notmuch_config_get (notmuch, NOTMUCH_CONFIG_PRIMARY_EMAIL)); if (strlen (response)) - notmuch_config_set_user_primary_email (config, response); + notmuch_conffile_set_user_primary_email (config, response); other_emails = g_ptr_array_new (); @@ -188,10 +188,10 @@ notmuch_setup_command (notmuch_database_t *notmuch, g_ptr_array_add (other_emails, talloc_strdup (config, response)); } while (strlen (response)); if (other_emails->len) - notmuch_config_set_user_other_email (config, - (const char **) - other_emails->pdata, - other_emails->len); + notmuch_conffile_set_user_other_email (config, + (const char **) + other_emails->pdata, + other_emails->len); g_ptr_array_free (other_emails, true); prompt ("Top-level directory of your email archive [%s]: ", @@ -200,7 +200,7 @@ notmuch_setup_command (notmuch_database_t *notmuch, const char *absolute_path; absolute_path = make_path_absolute (config, response); - notmuch_config_set_database_path (config, absolute_path); + notmuch_conffile_set_database_path (config, absolute_path); } new_tags = notmuch_config_get_values (notmuch, NOTMUCH_CONFIG_NEW_TAGS); @@ -212,8 +212,8 @@ notmuch_setup_command (notmuch_database_t *notmuch, if (strlen (response)) { GPtrArray *tags = parse_tag_list (config, response); - notmuch_config_set_new_tags (config, (const char **) tags->pdata, - tags->len); + notmuch_conffile_set_new_tags (config, (const char **) tags->pdata, + tags->len); g_ptr_array_free (tags, true); } @@ -227,20 +227,20 @@ notmuch_setup_command (notmuch_database_t *notmuch, if (strlen (response)) { GPtrArray *tags = parse_tag_list (config, response); - notmuch_config_set_search_exclude_tags (config, - (const char **) tags->pdata, - tags->len); + notmuch_conffile_set_search_exclude_tags (config, + (const char **) tags->pdata, + tags->len); g_ptr_array_free (tags, true); } - if (notmuch_config_save (config)) + if (notmuch_conffile_save (config)) return EXIT_FAILURE; if (config) - notmuch_config_close (config); + notmuch_conffile_close (config); - if (notmuch_config_is_new (config)) + if (notmuch_conffile_is_new (config)) welcome_message_post_setup (); return EXIT_SUCCESS; -- 2.30.1