]> granicus.if.org Git - neomutt/commitdiff
Improve const-correctness
authorFederico Kircheis <federico.kircheis@gmail.com>
Wed, 10 Jul 2019 18:30:30 +0000 (20:30 +0200)
committerRichard Russon <rich@flatcap.org>
Sat, 13 Jul 2019 23:10:35 +0000 (00:10 +0100)
of

  * convert_file_to
  * get_quote_level
  * mutt_write_one_header
  * classify_quote
  * gen_string_hash
  * gen_case_string_hash
  * mutt_buffer_enter_fname_full
  * parse_references

curs_lib.c
email/parse.c
hdrline.c
mutt/hash.c
pager.c
rfc3676.c
sendlib.c

index 64b9ff957a34fce27740483b2cfb230255d75b5e..486fed55afead6f5473b9cb9cf5da997b8424f30 100644 (file)
@@ -652,7 +652,7 @@ int mutt_buffer_enter_fname_full(const char *prompt, struct Buffer *fname,
   struct KeyEvent ch;
 
   SET_COLOR(MT_COLOR_PROMPT);
-  mutt_window_mvaddstr(MuttMessageWindow, 0, 0, (char *) prompt);
+  mutt_window_mvaddstr(MuttMessageWindow, 0, 0, prompt);
   addstr(_(" ('?' for list): "));
   NORMAL_COLOR;
   if (mutt_buffer_len(fname))
index 4745da6f76ce9f45547e67ac72c4e742a2f8adca..0214e2b73c33f86af254f2e25ec53ed62f31203b 100644 (file)
@@ -250,7 +250,7 @@ static void parse_content_disposition(const char *s, struct Body *ct)
  * @param head List to receive the references
  * @param s    String to parse
  */
-static void parse_references(struct ListHead *head, char *s)
+static void parse_references(struct ListHead *head, const char *s)
 {
   char *m = NULL;
   const char *sp = NULL;
index 96e85f8fc574a11d7243a93480d99820355b1f03..6b16b5b2bd8dfb6fc3b55003c355ba2c39ace00c 100644 (file)
--- a/hdrline.c
+++ b/hdrline.c
@@ -683,12 +683,11 @@ static const char *index_format_str(char *buf, size_t buflen, size_t col, int co
 
         if (optional && ((op == '[') || (op == '(')))
         {
-          char *is = NULL;
           now = time(NULL);
           struct tm tm = mutt_date_localtime(now);
           now -= (op == '(') ? e->received : e->date_sent;
 
-          is = (char *) prec;
+          char *is = (char *) prec;
           bool invert = false;
           if (*is == '>')
           {
index cc628beb2b09900bc108536f294939c58b2d8076..28607146b62548885e95fea15b845ae09ee58ee7 100644 (file)
@@ -45,7 +45,7 @@
 static size_t gen_string_hash(union HashKey key, size_t n)
 {
   size_t h = 0;
-  unsigned char *s = (unsigned char *) key.strkey;
+  const unsigned char *s = (const unsigned char *) key.strkey;
 
   while (*s)
     h += ((h << 7) + *s++);
@@ -76,7 +76,7 @@ static int cmp_string_key(union HashKey a, union HashKey b)
 static size_t gen_case_string_hash(union HashKey key, size_t n)
 {
   size_t h = 0;
-  unsigned char *s = (unsigned char *) key.strkey;
+  const unsigned char *s = (const unsigned char *) key.strkey;
 
   while (*s)
     h += ((h << 7) + tolower(*s++));
diff --git a/pager.c b/pager.c
index b059300f5564aa32c756989a91a738d505de8cd9..9288d79c453281aa253cced68bb2063ad4a561b9 100644 (file)
--- a/pager.c
+++ b/pager.c
@@ -524,7 +524,7 @@ static struct QClass *classify_quote(struct QClass **quote_list, const char *qpt
 {
   struct QClass *q_list = *quote_list;
   struct QClass *qc = NULL, *tmp = NULL, *ptr = NULL, *save = NULL;
-  char *tail_qptr = NULL;
+  const char *tail_qptr = NULL;
   int offset, tail_lng;
   int index = -1;
 
@@ -650,7 +650,7 @@ static struct QClass *classify_quote(struct QClass **quote_list, const char *qpt
 
         q_list = q_list->down;
         tail_lng = length - offset;
-        tail_qptr = (char *) qptr + offset;
+        tail_qptr = qptr + offset;
 
         while (q_list)
         {
@@ -752,7 +752,7 @@ static struct QClass *classify_quote(struct QClass **quote_list, const char *qpt
 
               q_list = q_list->down;
               tail_lng = length - offset;
-              tail_qptr = (char *) qptr + offset;
+              tail_qptr = qptr + offset;
 
               continue;
             }
index 46cc13bce8774fdb4436e888223e159da7a0f4d2..41000e96bd6886b5fb59019299bb9c2d727e00da 100644 (file)
--- a/rfc3676.c
+++ b/rfc3676.c
@@ -67,7 +67,7 @@ struct FlowedState
 static int get_quote_level(const char *line)
 {
   int quoted = 0;
-  char *p = (char *) line;
+  const char *p = line;
 
   while (p && (*p == '>'))
   {
index 887639e7309388db15bc7c23e3bb855c5f40f019..74c305017b9c3dd48ccb60836a30ce4cfc95c366 100644 (file)
--- a/sendlib.c
+++ b/sendlib.c
@@ -768,7 +768,7 @@ static void update_content_info(struct Content *info, struct ContentState *s,
  * in.
  */
 static size_t convert_file_to(FILE *fp, const char *fromcode, int ncodes,
-                              const char **tocodes, int *tocode, struct Content *info)
+                              char const *const *tocodes, int *tocode, struct Content *info)
 {
   char bufi[256], bufu[512], bufo[4 * sizeof(bufi)];
   size_t ret;
@@ -960,7 +960,7 @@ static size_t convert_file_from_to(FILE *fp, const char *fromcodes, const char *
         continue;
       fcode = mutt_str_substr_dup(c, c1);
 
-      ret = convert_file_to(fp, fcode, ncodes, (const char **) tcode, &cn, info);
+      ret = convert_file_to(fp, fcode, ncodes, (char const *const *) tcode, &cn, info);
       if (ret != (size_t)(-1))
       {
         *fromcode = fcode;
@@ -974,7 +974,7 @@ static size_t convert_file_from_to(FILE *fp, const char *fromcodes, const char *
   else
   {
     /* There is only one fromcode */
-    ret = convert_file_to(fp, fromcodes, ncodes, (const char **) tcode, &cn, info);
+    ret = convert_file_to(fp, fromcodes, ncodes, (char const *const *) tcode, &cn, info);
     if (ret != (size_t)(-1))
     {
       *tocode = tcode[cn];
@@ -2123,7 +2123,6 @@ static int write_one_header(FILE *fp, int pfxw, int max, int wraplen, const char
 int mutt_write_one_header(FILE *fp, const char *tag, const char *value,
                           const char *pfx, int wraplen, CopyHeaderFlags chflags)
 {
-  char *p = (char *) value;
   char *last = NULL, *line = NULL;
   int max = 0, w, rc = -1;
   int pfxw = mutt_strwidth(pfx);
@@ -2162,7 +2161,7 @@ int mutt_write_one_header(FILE *fp, const char *tag, const char *value,
     }
   }
 
-  p = v;
+  char *p = v;
   last = v;
   line = v;
   while (p && *p)