unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] cli: abstract common config get/set code
@ 2014-01-17 16:38 Jani Nikula
  2014-01-18 12:36 ` Tomi Ollila
  2014-01-19  0:14 ` David Bremner
  0 siblings, 2 replies; 4+ messages in thread
From: Jani Nikula @ 2014-01-17 16:38 UTC (permalink / raw)
  To: notmuch

Pretty straightforward abstraction similar to get/set list.

---

v2 of id:1376839205-5115-1-git-send-email-jani@nikula.org adding a few
comments about config value caching per David's request. Dropped the
2nd patch as too tricky.
---
 notmuch-config.c | 86 +++++++++++++++++++++++---------------------------------
 1 file changed, 35 insertions(+), 51 deletions(-)

diff --git a/notmuch-config.c b/notmuch-config.c
index 6845e3c..4aad9eb 100644
--- a/notmuch-config.c
+++ b/notmuch-config.c
@@ -496,6 +496,32 @@ notmuch_config_is_new (notmuch_config_t *config)
     return config->is_new;
 }
 
+static const char *
+_config_get (notmuch_config_t *config, char **field,
+	     const char *group, const char *key)
+{
+    /* read from config file and cache value, if not cached already */
+    if (*field == NULL) {
+	char *value;
+	value = g_key_file_get_string (config->key_file, group, key, NULL);
+	if (value) {
+	    *field = talloc_strdup (config, value);
+	    free (value);
+	}
+    }
+    return *field;
+}
+
+static void
+_config_set (notmuch_config_t *config, char **field,
+	     const char *group, const char *key, const char *value)
+{
+    g_key_file_set_string (config->key_file, group, key, value);
+
+    /* drop the cached value */
+    talloc_free (*field);
+    *field = NULL;
+}
 
 static const char **
 _config_get_list (notmuch_config_t *config,
@@ -504,6 +530,7 @@ _config_get_list (notmuch_config_t *config,
 {
     assert(outlist);
 
+    /* read from config file and cache value, if not cached already */
     if (*outlist == NULL) {
 
 	char **inlist = g_key_file_get_string_list (config->key_file,
@@ -535,6 +562,8 @@ _config_set_list (notmuch_config_t *config,
 		  size_t length, const char ***config_var )
 {
     g_key_file_set_string_list (config->key_file, group, name, list, length);
+
+    /* drop the cached value */
     talloc_free (*config_var);
     *config_var = NULL;
 }
@@ -542,85 +571,40 @@ _config_set_list (notmuch_config_t *config,
 const char *
 notmuch_config_get_database_path (notmuch_config_t *config)
 {
-    char *path;
-
-    if (config->database_path == NULL) {
-	path = g_key_file_get_string (config->key_file,
-				      "database", "path", NULL);
-	if (path) {
-	    config->database_path = talloc_strdup (config, path);
-	    free (path);
-	}
-    }
-
-    return config->database_path;
+    return _config_get (config, &config->database_path, "database", "path");
 }
 
 void
 notmuch_config_set_database_path (notmuch_config_t *config,
 				  const char *database_path)
 {
-    g_key_file_set_string (config->key_file,
-			   "database", "path", database_path);
-
-    talloc_free (config->database_path);
-    config->database_path = NULL;
+    _config_set (config, &config->database_path, "database", "path", database_path);
 }
 
 const char *
 notmuch_config_get_user_name (notmuch_config_t *config)
 {
-    char *name;
-
-    if (config->user_name == NULL) {
-	name = g_key_file_get_string (config->key_file,
-				      "user", "name", NULL);
-	if (name) {
-	    config->user_name = talloc_strdup (config, name);
-	    free (name);
-	}
-    }
-
-    return config->user_name;
+    return _config_get (config, &config->user_name, "user", "name");
 }
 
 void
 notmuch_config_set_user_name (notmuch_config_t *config,
 			      const char *user_name)
 {
-    g_key_file_set_string (config->key_file,
-			   "user", "name", user_name);
-
-    talloc_free (config->user_name);
-    config->user_name = NULL;
+    _config_set (config, &config->user_name, "user", "name", user_name);
 }
 
 const char *
 notmuch_config_get_user_primary_email (notmuch_config_t *config)
 {
-    char *email;
-
-    if (config->user_primary_email == NULL) {
-	email = g_key_file_get_string (config->key_file,
-				       "user", "primary_email", NULL);
-	if (email) {
-	    config->user_primary_email = talloc_strdup (config, email);
-	    free (email);
-	}
-    }
-
-    return config->user_primary_email;
+    return _config_get (config, &config->user_primary_email, "user", "primary_email");
 }
 
 void
 notmuch_config_set_user_primary_email (notmuch_config_t *config,
 				       const char *primary_email)
 {
-    g_key_file_set_string (config->key_file,
-			   "user", "primary_email", primary_email);
-
-    talloc_free (config->user_primary_email);
-    config->user_primary_email = NULL;
+    _config_set (config, &config->user_primary_email, "user", "primary_email", primary_email);
 }
 
 const char **
-- 
1.8.5.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] cli: abstract common config get/set code
  2014-01-17 16:38 [PATCH] cli: abstract common config get/set code Jani Nikula
@ 2014-01-18 12:36 ` Tomi Ollila
  2014-01-18 13:36   ` Mark Walters
  2014-01-19  0:14 ` David Bremner
  1 sibling, 1 reply; 4+ messages in thread
From: Tomi Ollila @ 2014-01-18 12:36 UTC (permalink / raw)
  To: Jani Nikula, notmuch

On Fri, Jan 17 2014, Jani Nikula <jani@nikula.org> wrote:

> Pretty straightforward abstraction similar to get/set list.
>
> ---

LGTM. tests pass.

>
> v2 of id:1376839205-5115-1-git-send-email-jani@nikula.org adding a few
> comments about config value caching per David's request. Dropped the
> 2nd patch as too tricky.
> ---
>  notmuch-config.c | 86 +++++++++++++++++++++++---------------------------------
>  1 file changed, 35 insertions(+), 51 deletions(-)
>
> diff --git a/notmuch-config.c b/notmuch-config.c
> index 6845e3c..4aad9eb 100644
> --- a/notmuch-config.c
> +++ b/notmuch-config.c
> @@ -496,6 +496,32 @@ notmuch_config_is_new (notmuch_config_t *config)
>      return config->is_new;
>  }
>
> +static const char *
> +_config_get (notmuch_config_t *config, char **field,
> +          const char *group, const char *key)
> +{
> +    /* read from config file and cache value, if not cached already */
> +    if (*field == NULL) {
> +     char *value;
> +     value = g_key_file_get_string (config->key_file, group, key, NULL);
> +     if (value) {
> +         *field = talloc_strdup (config, value);
> +         free (value);
> +     }
> +    }
> +    return *field;
> +}
> +
> +static void
> +_config_set (notmuch_config_t *config, char **field,
> +          const char *group, const char *key, const char *value)
> +{
> +    g_key_file_set_string (config->key_file, group, key, value);
> +
> +    /* drop the cached value */
> +    talloc_free (*field);
> +    *field = NULL;
> +}
>
>  static const char **
>  _config_get_list (notmuch_config_t *config,
> @@ -504,6 +530,7 @@ _config_get_list (notmuch_config_t *config,
>  {
>      assert(outlist);
>
> +    /* read from config file and cache value, if not cached already */
>      if (*outlist == NULL) {
>
>       char **inlist = g_key_file_get_string_list (config->key_file,
> @@ -535,6 +562,8 @@ _config_set_list (notmuch_config_t *config,
>                 size_t length, const char ***config_var )
>  {
>      g_key_file_set_string_list (config->key_file, group, name, list, length);
> +
> +    /* drop the cached value */
>      talloc_free (*config_var);
>      *config_var = NULL;
>  }
> @@ -542,85 +571,40 @@ _config_set_list (notmuch_config_t *config,
>  const char *
>  notmuch_config_get_database_path (notmuch_config_t *config)
>  {
> -    char *path;
> -
> -    if (config->database_path == NULL) {
> -     path = g_key_file_get_string (config->key_file,
> -                                   "database", "path", NULL);
> -     if (path) {
> -         config->database_path = talloc_strdup (config, path);
> -         free (path);
> -     }
> -    }
> -
> -    return config->database_path;
> +    return _config_get (config, &config->database_path, "database", "path");
>  }
>
>  void
>  notmuch_config_set_database_path (notmuch_config_t *config,
>                                 const char *database_path)
>  {
> -    g_key_file_set_string (config->key_file,
> -                        "database", "path", database_path);
> -
> -    talloc_free (config->database_path);
> -    config->database_path = NULL;
> +    _config_set (config, &config->database_path, "database", "path", database_path);
>  }
>
>  const char *
>  notmuch_config_get_user_name (notmuch_config_t *config)
>  {
> -    char *name;
> -
> -    if (config->user_name == NULL) {
> -     name = g_key_file_get_string (config->key_file,
> -                                   "user", "name", NULL);
> -     if (name) {
> -         config->user_name = talloc_strdup (config, name);
> -         free (name);
> -     }
> -    }
> -
> -    return config->user_name;
> +    return _config_get (config, &config->user_name, "user", "name");
>  }
>
>  void
>  notmuch_config_set_user_name (notmuch_config_t *config,
>                             const char *user_name)
>  {
> -    g_key_file_set_string (config->key_file,
> -                        "user", "name", user_name);
> -
> -    talloc_free (config->user_name);
> -    config->user_name = NULL;
> +    _config_set (config, &config->user_name, "user", "name", user_name);
>  }
>
>  const char *
>  notmuch_config_get_user_primary_email (notmuch_config_t *config)
>  {
> -    char *email;
> -
> -    if (config->user_primary_email == NULL) {
> -     email = g_key_file_get_string (config->key_file,
> -                                    "user", "primary_email", NULL);
> -     if (email) {
> -         config->user_primary_email = talloc_strdup (config, email);
> -         free (email);
> -     }
> -    }
> -
> -    return config->user_primary_email;
> +    return _config_get (config, &config->user_primary_email, "user", "primary_email");
>  }
>
>  void
>  notmuch_config_set_user_primary_email (notmuch_config_t *config,
>                                      const char *primary_email)
>  {
> -    g_key_file_set_string (config->key_file,
> -                        "user", "primary_email", primary_email);
> -
> -    talloc_free (config->user_primary_email);
> -    config->user_primary_email = NULL;
> +    _config_set (config, &config->user_primary_email, "user", "primary_email", primary_email);
>  }
>
>  const char **
> --
> 1.8.5.2
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] cli: abstract common config get/set code
  2014-01-18 12:36 ` Tomi Ollila
@ 2014-01-18 13:36   ` Mark Walters
  0 siblings, 0 replies; 4+ messages in thread
From: Mark Walters @ 2014-01-18 13:36 UTC (permalink / raw)
  To: Tomi Ollila, Jani Nikula, notmuch


LGTM too

Mark

On Sat, 18 Jan 2014, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> On Fri, Jan 17 2014, Jani Nikula <jani@nikula.org> wrote:
>
>> Pretty straightforward abstraction similar to get/set list.
>>
>> ---
>
> LGTM. tests pass.
>
>>
>> v2 of id:1376839205-5115-1-git-send-email-jani@nikula.org adding a few
>> comments about config value caching per David's request. Dropped the
>> 2nd patch as too tricky.
>> ---
>>  notmuch-config.c | 86 +++++++++++++++++++++++---------------------------------
>>  1 file changed, 35 insertions(+), 51 deletions(-)
>>
>> diff --git a/notmuch-config.c b/notmuch-config.c
>> index 6845e3c..4aad9eb 100644
>> --- a/notmuch-config.c
>> +++ b/notmuch-config.c
>> @@ -496,6 +496,32 @@ notmuch_config_is_new (notmuch_config_t *config)
>>      return config->is_new;
>>  }
>>
>> +static const char *
>> +_config_get (notmuch_config_t *config, char **field,
>> +          const char *group, const char *key)
>> +{
>> +    /* read from config file and cache value, if not cached already */
>> +    if (*field == NULL) {
>> +     char *value;
>> +     value = g_key_file_get_string (config->key_file, group, key, NULL);
>> +     if (value) {
>> +         *field = talloc_strdup (config, value);
>> +         free (value);
>> +     }
>> +    }
>> +    return *field;
>> +}
>> +
>> +static void
>> +_config_set (notmuch_config_t *config, char **field,
>> +          const char *group, const char *key, const char *value)
>> +{
>> +    g_key_file_set_string (config->key_file, group, key, value);
>> +
>> +    /* drop the cached value */
>> +    talloc_free (*field);
>> +    *field = NULL;
>> +}
>>
>>  static const char **
>>  _config_get_list (notmuch_config_t *config,
>> @@ -504,6 +530,7 @@ _config_get_list (notmuch_config_t *config,
>>  {
>>      assert(outlist);
>>
>> +    /* read from config file and cache value, if not cached already */
>>      if (*outlist == NULL) {
>>
>>       char **inlist = g_key_file_get_string_list (config->key_file,
>> @@ -535,6 +562,8 @@ _config_set_list (notmuch_config_t *config,
>>                 size_t length, const char ***config_var )
>>  {
>>      g_key_file_set_string_list (config->key_file, group, name, list, length);
>> +
>> +    /* drop the cached value */
>>      talloc_free (*config_var);
>>      *config_var = NULL;
>>  }
>> @@ -542,85 +571,40 @@ _config_set_list (notmuch_config_t *config,
>>  const char *
>>  notmuch_config_get_database_path (notmuch_config_t *config)
>>  {
>> -    char *path;
>> -
>> -    if (config->database_path == NULL) {
>> -     path = g_key_file_get_string (config->key_file,
>> -                                   "database", "path", NULL);
>> -     if (path) {
>> -         config->database_path = talloc_strdup (config, path);
>> -         free (path);
>> -     }
>> -    }
>> -
>> -    return config->database_path;
>> +    return _config_get (config, &config->database_path, "database", "path");
>>  }
>>
>>  void
>>  notmuch_config_set_database_path (notmuch_config_t *config,
>>                                 const char *database_path)
>>  {
>> -    g_key_file_set_string (config->key_file,
>> -                        "database", "path", database_path);
>> -
>> -    talloc_free (config->database_path);
>> -    config->database_path = NULL;
>> +    _config_set (config, &config->database_path, "database", "path", database_path);
>>  }
>>
>>  const char *
>>  notmuch_config_get_user_name (notmuch_config_t *config)
>>  {
>> -    char *name;
>> -
>> -    if (config->user_name == NULL) {
>> -     name = g_key_file_get_string (config->key_file,
>> -                                   "user", "name", NULL);
>> -     if (name) {
>> -         config->user_name = talloc_strdup (config, name);
>> -         free (name);
>> -     }
>> -    }
>> -
>> -    return config->user_name;
>> +    return _config_get (config, &config->user_name, "user", "name");
>>  }
>>
>>  void
>>  notmuch_config_set_user_name (notmuch_config_t *config,
>>                             const char *user_name)
>>  {
>> -    g_key_file_set_string (config->key_file,
>> -                        "user", "name", user_name);
>> -
>> -    talloc_free (config->user_name);
>> -    config->user_name = NULL;
>> +    _config_set (config, &config->user_name, "user", "name", user_name);
>>  }
>>
>>  const char *
>>  notmuch_config_get_user_primary_email (notmuch_config_t *config)
>>  {
>> -    char *email;
>> -
>> -    if (config->user_primary_email == NULL) {
>> -     email = g_key_file_get_string (config->key_file,
>> -                                    "user", "primary_email", NULL);
>> -     if (email) {
>> -         config->user_primary_email = talloc_strdup (config, email);
>> -         free (email);
>> -     }
>> -    }
>> -
>> -    return config->user_primary_email;
>> +    return _config_get (config, &config->user_primary_email, "user", "primary_email");
>>  }
>>
>>  void
>>  notmuch_config_set_user_primary_email (notmuch_config_t *config,
>>                                      const char *primary_email)
>>  {
>> -    g_key_file_set_string (config->key_file,
>> -                        "user", "primary_email", primary_email);
>> -
>> -    talloc_free (config->user_primary_email);
>> -    config->user_primary_email = NULL;
>> +    _config_set (config, &config->user_primary_email, "user", "primary_email", primary_email);
>>  }
>>
>>  const char **
>> --
>> 1.8.5.2
>>
>> _______________________________________________
>> notmuch mailing list
>> notmuch@notmuchmail.org
>> http://notmuchmail.org/mailman/listinfo/notmuch
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] cli: abstract common config get/set code
  2014-01-17 16:38 [PATCH] cli: abstract common config get/set code Jani Nikula
  2014-01-18 12:36 ` Tomi Ollila
@ 2014-01-19  0:14 ` David Bremner
  1 sibling, 0 replies; 4+ messages in thread
From: David Bremner @ 2014-01-19  0:14 UTC (permalink / raw)
  To: Jani Nikula, notmuch

Jani Nikula <jani@nikula.org> writes:

> Pretty straightforward abstraction similar to get/set list.
>

pushed.

d

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-01-19  0:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-17 16:38 [PATCH] cli: abstract common config get/set code Jani Nikula
2014-01-18 12:36 ` Tomi Ollila
2014-01-18 13:36   ` Mark Walters
2014-01-19  0:14 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).