config: don't exclude testable cases

diff --git a/config/address.c b/config/address.c
index 5e0796fa3356b91c4590cce31bc5a5a8f08e9a14..8fe31f740e1ce2ce0c6af4b3d597ff3ff09e29e7 100644 (file)
--- a/config/address.c
+++ b/config/address.c
@@ -272,7 +272,7 @@ struct Address *address_new(const char *addr)
 void address_free(struct Address **addr)
 {
   if (!addr || !*addr)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   FREE(&(*addr)->personal);
   FREE(&(*addr)->mailbox);

diff --git a/config/bool.c b/config/bool.c
index 8323c47beee41bce893bef6fcdd75c58c8391e8f..fe379c98be7d18c30cddf5f5bd378f6b0a2eaca2 100644 (file)
--- a/config/bool.c
+++ b/config/bool.c
@@ -212,7 +212,7 @@ void bool_init(struct ConfigSet *cs)
 int bool_he_toggle(struct ConfigSet *cs, struct HashElem *he, struct Buffer *err)
 {
   if (!cs || !he || !he->data)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   if (DTYPE(he->type) != DT_BOOL)
     return CSR_ERR_CODE;
@@ -243,7 +243,7 @@ int bool_he_toggle(struct ConfigSet *cs, struct HashElem *he, struct Buffer *err
 int bool_str_toggle(struct ConfigSet *cs, const char *name, struct Buffer *err)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)

diff --git a/config/cfgaccount.c b/config/cfgaccount.c
index 100dd5596b196c82320ebe7fd9862e48ebee3892..6f5f17e21a49791e41856ffd2235080e61b1a1bb 100644 (file)
--- a/config/cfgaccount.c
+++ b/config/cfgaccount.c
@@ -46,7 +46,7 @@ struct CfgAccount *ac_new(const struct ConfigSet *cs, const char *name,
                           const char *var_names[])
 {
   if (!cs || !name || !var_names)
-    return NULL; /* LCOV_EXCL_LINE */
+    return NULL;
 
   int count = 0;
   for (; var_names[count]; count++)
@@ -97,7 +97,7 @@ struct CfgAccount *ac_new(const struct ConfigSet *cs, const char *name,
 void ac_free(const struct ConfigSet *cs, struct CfgAccount **ac)
 {
   if (!cs || !ac || !*ac)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   char child[128];
   struct Buffer *err = mutt_buffer_pool_get();
@@ -129,7 +129,7 @@ void ac_free(const struct ConfigSet *cs, struct CfgAccount **ac)
 int ac_set_value(const struct CfgAccount *ac, size_t vid, intptr_t value, struct Buffer *err)
 {
   if (!ac)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
   if (vid >= ac->num_vars)
     return CSR_ERR_UNKNOWN;
 
@@ -147,7 +147,7 @@ int ac_set_value(const struct CfgAccount *ac, size_t vid, intptr_t value, struct
 int ac_get_value(const struct CfgAccount *ac, size_t vid, struct Buffer *result)
 {
   if (!ac)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
   if (vid >= ac->num_vars)
     return CSR_ERR_UNKNOWN;
 

diff --git a/config/mbtable.c b/config/mbtable.c
index b9524bdb91865f379e6d16933e737a5d55fa5213..6d316f84c2aa084455bdf6ef642ee0f874531689 100644 (file)
--- a/config/mbtable.c
+++ b/config/mbtable.c
@@ -307,7 +307,7 @@ void mbtable_init(struct ConfigSet *cs)
 void mbtable_free(struct MbTable **table)
 {
   if (!table || !*table)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   FREE(&(*table)->orig_str);
   FREE(&(*table)->chars);

diff --git a/config/quad.c b/config/quad.c
index 3cb250ade5c3fb427b5d89979230903c79d1fa1c..1ab8d28e858a6fa8b16c732d3ae2affb62425fae 100644 (file)
--- a/config/quad.c
+++ b/config/quad.c
@@ -225,7 +225,7 @@ static int quad_toggle(int opt)
 int quad_he_toggle(struct ConfigSet *cs, struct HashElem *he, struct Buffer *err)
 {
   if (!cs || !he || !he->data)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   if (DTYPE(he->type) != DT_QUAD)
     return CSR_ERR_CODE;

diff --git a/config/regex.c b/config/regex.c
index d3923f8068f23e97fa6314e25e345fb03af4b666..60774e81b0b82cc337fdd79aaf2840ef2d301edb 100644 (file)
--- a/config/regex.c
+++ b/config/regex.c
@@ -272,7 +272,7 @@ void regex_init(struct ConfigSet *cs)
 struct Regex *regex_new(const char *str, int flags, struct Buffer *err)
 {
   if (!str)
-    return NULL; /* LCOV_EXCL_LINE */
+    return NULL;
 
   int rflags = 0;
   struct Regex *reg = mutt_mem_calloc(1, sizeof(struct Regex));
@@ -312,7 +312,7 @@ struct Regex *regex_new(const char *str, int flags, struct Buffer *err)
 void regex_free(struct Regex **r)
 {
   if (!r || !*r)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   FREE(&(*r)->pattern);
   if ((*r)->regex)

diff --git a/config/set.c b/config/set.c
index 2883371996aadb85d7fbd15edf3561b736dd8185..a99809847f74f846048c6f0de81db8068ee2321c 100644 (file)
--- a/config/set.c
+++ b/config/set.c
@@ -99,7 +99,7 @@ static struct HashElem *create_synonym(const struct ConfigSet *cs,
   struct HashElem *child =
       mutt_hash_typed_insert(cs->hash, cdef->name, cdef->type, (void *) cdef);
   if (!child)
-    return NULL;
+    return NULL; /* LCOV_EXCL_LINE */
 
   cdef->var = parent;
   return child;
@@ -131,7 +131,7 @@ static struct HashElem *reg_one_var(const struct ConfigSet *cs,
   struct HashElem *he =
       mutt_hash_typed_insert(cs->hash, cdef->name, cdef->type, (void *) cdef);
   if (!he)
-    return NULL;
+    return NULL; /* LCOV_EXCL_LINE */
 
   if (cst && cst->reset)
     cst->reset(cs, cdef->var, cdef, err);
@@ -159,7 +159,7 @@ struct ConfigSet *cs_new(size_t size)
 void cs_init(struct ConfigSet *cs, size_t size)
 {
   if (!cs)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   memset(cs, 0, sizeof(*cs));
   cs->hash = mutt_hash_new(size, MUTT_HASH_NO_FLAGS);
@@ -173,7 +173,7 @@ void cs_init(struct ConfigSet *cs, size_t size)
 void cs_free(struct ConfigSet **cs)
 {
   if (!cs || !*cs)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   mutt_hash_free(&(*cs)->hash);
   FREE(cs);
@@ -188,7 +188,7 @@ void cs_free(struct ConfigSet **cs)
 struct HashElem *cs_get_elem(const struct ConfigSet *cs, const char *name)
 {
   if (!cs || !name)
-    return NULL; /* LCOV_EXCL_LINE */
+    return NULL;
 
   struct HashElem *he = mutt_hash_find_elem(cs->hash, name);
   if (!he)
@@ -211,7 +211,7 @@ struct HashElem *cs_get_elem(const struct ConfigSet *cs, const char *name)
 const struct ConfigSetType *cs_get_type_def(const struct ConfigSet *cs, unsigned int type)
 {
   if (!cs)
-    return NULL; /* LCOV_EXCL_LINE */
+    return NULL;
 
   type = DTYPE(type);
   if ((type < 1) || (type >= mutt_array_size(cs->types)))
@@ -233,7 +233,7 @@ const struct ConfigSetType *cs_get_type_def(const struct ConfigSet *cs, unsigned
 bool cs_register_type(struct ConfigSet *cs, unsigned int type, const struct ConfigSetType *cst)
 {
   if (!cs || !cst)
-    return false; /* LCOV_EXCL_LINE */
+    return false;
 
   if (!cst->name || !cst->string_set || !cst->string_get || !cst->reset ||
       !cst->native_set || !cst->native_get)
@@ -261,7 +261,7 @@ bool cs_register_type(struct ConfigSet *cs, unsigned int type, const struct Conf
 bool cs_register_variables(const struct ConfigSet *cs, struct ConfigDef vars[], int flags)
 {
   if (!cs || !vars)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct Buffer *err = mutt_buffer_pool_get();
 
@@ -291,7 +291,7 @@ struct HashElem *cs_inherit_variable(const struct ConfigSet *cs,
                                      struct HashElem *parent, const char *name)
 {
   if (!cs || !parent)
-    return NULL; /* LCOV_EXCL_LINE */
+    return NULL;
 
   struct Inheritance *i = mutt_mem_calloc(1, sizeof(*i));
   i->parent = parent;
@@ -315,7 +315,7 @@ struct HashElem *cs_inherit_variable(const struct ConfigSet *cs,
 void cs_add_listener(struct ConfigSet *cs, cs_listener fn)
 {
   if (!cs || !fn)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   for (size_t i = 0; i < mutt_array_size(cs->listeners); i++)
   {
@@ -344,7 +344,7 @@ void cs_add_listener(struct ConfigSet *cs, cs_listener fn)
 void cs_remove_listener(struct ConfigSet *cs, cs_listener fn)
 {
   if (!cs || !fn)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   for (size_t i = 0; i < mutt_array_size(cs->listeners); i++)
   {
@@ -368,7 +368,7 @@ void cs_notify_listeners(const struct ConfigSet *cs, struct HashElem *he,
                          const char *name, enum ConfigEvent ev)
 {
   if (!cs || !he || !name)
-    return; /* LCOV_EXCL_LINE */
+    return;
 
   for (size_t i = 0; i < mutt_array_size(cs->listeners); i++)
   {
@@ -389,7 +389,7 @@ void cs_notify_listeners(const struct ConfigSet *cs, struct HashElem *he,
 int cs_he_reset(const struct ConfigSet *cs, struct HashElem *he, struct Buffer *err)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   /* An inherited var that's already pointing to its parent.
    * Return 'success', but don't send a notification. */
@@ -436,7 +436,7 @@ int cs_he_reset(const struct ConfigSet *cs, struct HashElem *he, struct Buffer *
 int cs_str_reset(const struct ConfigSet *cs, const char *name, struct Buffer *err)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -460,7 +460,7 @@ int cs_he_initial_set(const struct ConfigSet *cs, struct HashElem *he,
                       const char *value, struct Buffer *err)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct ConfigDef *cdef = NULL;
   const struct ConfigSetType *cst = NULL;
@@ -501,7 +501,7 @@ int cs_str_initial_set(const struct ConfigSet *cs, const char *name,
                        const char *value, struct Buffer *err)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -526,7 +526,7 @@ int cs_str_initial_set(const struct ConfigSet *cs, const char *name,
 int cs_he_initial_get(const struct ConfigSet *cs, struct HashElem *he, struct Buffer *result)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct Inheritance *i = NULL;
   const struct ConfigDef *cdef = NULL;
@@ -567,7 +567,7 @@ int cs_he_initial_get(const struct ConfigSet *cs, struct HashElem *he, struct Bu
 int cs_str_initial_get(const struct ConfigSet *cs, const char *name, struct Buffer *result)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -591,7 +591,7 @@ int cs_he_string_set(const struct ConfigSet *cs, struct HashElem *he,
                      const char *value, struct Buffer *err)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct ConfigDef *cdef = NULL;
   const struct ConfigSetType *cst = NULL;
@@ -618,7 +618,7 @@ int cs_he_string_set(const struct ConfigSet *cs, struct HashElem *he,
   }
 
   if (!var)
-    return CSR_ERR_CODE;
+    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
 
   int rc = cst->string_set(cs, var, cdef, value, err);
   if (CSR_RESULT(rc) != CSR_SUCCESS)
@@ -646,7 +646,7 @@ int cs_str_string_set(const struct ConfigSet *cs, const char *name,
                       const char *value, struct Buffer *err)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -668,7 +668,7 @@ int cs_str_string_set(const struct ConfigSet *cs, const char *name,
 int cs_he_string_get(const struct ConfigSet *cs, struct HashElem *he, struct Buffer *result)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct Inheritance *i = NULL;
   const struct ConfigDef *cdef = NULL;
@@ -716,7 +716,7 @@ int cs_he_string_get(const struct ConfigSet *cs, struct HashElem *he, struct Buf
 int cs_str_string_get(const struct ConfigSet *cs, const char *name, struct Buffer *result)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -740,7 +740,7 @@ int cs_he_native_set(const struct ConfigSet *cs, struct HashElem *he,
                      intptr_t value, struct Buffer *err)
 {
   if (!cs || !he)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   const struct ConfigDef *cdef = NULL;
   const struct ConfigSetType *cst = NULL;
@@ -790,7 +790,7 @@ int cs_str_native_set(const struct ConfigSet *cs, const char *name,
                       intptr_t value, struct Buffer *err)
 {
   if (!cs || !name)
-    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
+    return CSR_ERR_CODE;
 
   struct HashElem *he = cs_get_elem(cs, name);
   if (!he)
@@ -818,10 +818,7 @@ int cs_str_native_set(const struct ConfigSet *cs, const char *name,
   }
 
   if (!cst)
-  {
-    mutt_debug(LL_DEBUG1, "Variable '%s' has an invalid type %d\n", cdef->name, he->type);
-    return CSR_ERR_CODE;
-  }
+    return CSR_ERR_CODE; /* LCOV_EXCL_LINE */
 
   int rc = cst->native_set(cs, var, cdef, value, err);
   if (CSR_RESULT(rc) == CSR_SUCCESS)
@@ -845,7 +842,7 @@ int cs_str_native_set(const struct ConfigSet *cs, const char *name,
 intptr_t cs_he_native_get(const struct ConfigSet *cs, struct HashElem *he, struct Buffer *err)
 {
   if (!cs || !he)
-    return INT_MIN; /* LCOV_EXCL_LINE */
+    return INT_MIN;
 
   struct Inheritance *i = NULL;
   const struct ConfigDef *cdef = NULL;
@@ -892,7 +889,7 @@ intptr_t cs_he_native_get(const struct ConfigSet *cs, struct HashElem *he, struc
 intptr_t cs_str_native_get(const struct ConfigSet *cs, const char *name, struct Buffer *err)
 {
   if (!cs || !name)
-    return INT_MIN; /* LCOV_EXCL_LINE */
+    return INT_MIN;
 
   struct HashElem *he = cs_get_elem(cs, name);
   return cs_he_native_get(cs, he, err);