scan-build: fix errors
authorRichard Russon <rich@flatcap.org>
Fri, 5 Oct 2018 17:14:09 +0000 (18:14 +0100)
committerRichard Russon <rich@flatcap.org>
Fri, 5 Oct 2018 17:14:09 +0000 (18:14 +0100)
compose.c
edit.c
test/config/account.c
test/config/quad.c

index 53b1dca3c10e2a3fea6470fe2ce3503a9dbf978c..665081f282eeab01e453162abc307da4901898ea 100644 (file)
--- a/compose.c
+++ b/compose.c
@@ -1255,9 +1255,8 @@ int mutt_compose_menu(struct Email *msg, char *fcc, size_t fcclen, struct Email
         }
 
         /* traverse to see whether all the parts have Content-Language: set */
-        struct Body *b = msg->content;
         int tagged_with_lang_num = 0;
-        for (i = 0; b; b = b->next)
+        for (struct Body *b = msg->content; b; b = b->next)
           if (b->tagged && b->language && *b->language)
             tagged_with_lang_num++;
 
diff --git a/edit.c b/edit.c
index 3c0d7c14793efabf41d5f05dc09819c7d2de4416..f9302bdc0b0e5632d699af6594fe45a3b89a3a69 100644 (file)
--- a/edit.c
+++ b/edit.c
@@ -203,7 +203,8 @@ static void be_free_memory(char **buf, int buflen)
 static char **be_include_messages(char *msg, char **buf, int *bufmax,
                                   int *buflen, int pfx, int inc_hdrs)
 {
-  int offset, bytes, n;
+  int n;
+  // int offset, bytes;
   char tmp[LONG_STRING];
 
   if (!msg || !buf || !bufmax || !buflen)
@@ -229,6 +230,9 @@ static char **be_include_messages(char *msg, char **buf, int *bufmax,
         mutt_mem_realloc(&buf, sizeof(char *) * (*bufmax += 25));
       buf[(*buflen)++] = mutt_str_strdup(tmp);
 
+#if 0
+      /* This only worked for mbox Mailboxes because they had Context->fp set.
+       * As that no longer exists, the code is now completely broken. */
       bytes = Context->mailbox->hdrs[n]->content->length;
       if (inc_hdrs)
       {
@@ -237,9 +241,8 @@ static char **be_include_messages(char *msg, char **buf, int *bufmax,
       }
       else
         offset = Context->mailbox->hdrs[n]->content->offset;
-      /* This only worked for mbox Mailboxes because they had Context->fp set.
-       * As that no longer exists, the code is now completely broken. */
-      // buf = be_snarf_data(Context->fp, buf, bufmax, buflen, offset, bytes, pfx);
+      buf = be_snarf_data(Context->fp, buf, bufmax, buflen, offset, bytes, pfx);
+#endif
 
       if (*bufmax == *buflen)
         mutt_mem_realloc(&buf, sizeof(char *) * (*bufmax += 25));
index e83795237ef776161353be086bcf88f73a55b156..c634f480ede7eb25178500ca03ec67702719b73a 100644 (file)
@@ -164,8 +164,8 @@ void config_account(void)
 
   const char *name = "fruit:Apple";
   mutt_buffer_reset(&err);
-  int result = cs_str_string_get(cs, name, &err);
-  if (TEST_CHECK(CSR_RESULT(result) == CSR_SUCCESS))
+  rc = cs_str_string_get(cs, name, &err);
+  if (TEST_CHECK(CSR_RESULT(rc) == CSR_SUCCESS))
   {
     TEST_MSG("%s = '%s'\n", name, err.data);
   }
@@ -176,8 +176,8 @@ void config_account(void)
   }
 
   mutt_buffer_reset(&err);
-  result = cs_str_native_set(cs, name, 42, &err);
-  if (TEST_CHECK(CSR_RESULT(result) == CSR_SUCCESS))
+  rc = cs_str_native_set(cs, name, 42, &err);
+  if (TEST_CHECK(CSR_RESULT(rc) == CSR_SUCCESS))
   {
     TEST_MSG("Set %s\n", name);
   }
@@ -192,19 +192,15 @@ void config_account(void)
     return;
 
   mutt_buffer_reset(&err);
-  result = cs_str_initial_set(cs, name, "42", &err);
+  rc = cs_str_initial_set(cs, name, "42", &err);
   if (TEST_CHECK(CSR_RESULT(rc) != CSR_SUCCESS))
   {
-    TEST_MSG("Expected error\n");
-  }
-  else
-  {
-    TEST_MSG("This test should have failed\n");
+    TEST_MSG("%s\n", err.data);
     return;
   }
 
   mutt_buffer_reset(&err);
-  result = cs_str_initial_get(cs, name, &err);
+  rc = cs_str_initial_get(cs, name, &err);
   if (TEST_CHECK(CSR_RESULT(rc) != CSR_SUCCESS))
   {
     TEST_MSG("Expected error\n");
@@ -221,8 +217,8 @@ void config_account(void)
     return;
 
   mutt_buffer_reset(&err);
-  result = cs_he_native_set(cs, he, 42, &err);
-  if (TEST_CHECK(CSR_RESULT(result) == CSR_SUCCESS))
+  rc = cs_he_native_set(cs, he, 42, &err);
+  if (TEST_CHECK(CSR_RESULT(rc) == CSR_SUCCESS))
   {
     TEST_MSG("Set %s\n", name);
   }
index 4ef038268ddbb8a2049a5144469063370f227aeb..a5b1952ddbda855b5d167c932d71071fcc025848 100644 (file)
@@ -243,7 +243,7 @@ static bool test_string_get(struct ConfigSet *cs, struct Buffer *err)
   int rc;
   for (unsigned int i = 0; i < mutt_array_size(valid); i++)
   {
-    VarElderberry = i;
+    VarElderberry = valid[i];
     mutt_buffer_reset(err);
     rc = cs_str_string_get(cs, name, err);
     if (CSR_RESULT(rc) != CSR_SUCCESS)