]> granicus.if.org Git - neomutt/commitdiff
return early where possible
authorRichard Russon <rich@flatcap.org>
Mon, 4 Dec 2017 23:46:04 +0000 (23:46 +0000)
committerRichard Russon <rich@flatcap.org>
Thu, 7 Dec 2017 12:20:51 +0000 (12:20 +0000)
init.c

diff --git a/init.c b/init.c
index 7d9580376ebcbdacc535e6997108ccd8920ba339..e0d0cde10bb38f947525895ea2bdf7211f6b32ff 100644 (file)
--- a/init.c
+++ b/init.c
@@ -1150,11 +1150,11 @@ static int parse_unstailq(struct Buffer *buf, struct Buffer *s,
 
 static void alternates_clean(void)
 {
-  if (Context && Context->msgcount)
-  {
-    for (int i = 0; i < Context->msgcount; i++)
-      Context->hdrs[i]->recip_valid = false;
-  }
+  if (!Context)
+    return;
+
+  for (int i = 0; i < Context->msgcount; i++)
+    Context->hdrs[i]->recip_valid = false;
 }
 
 static int parse_alternates(struct Buffer *buf, struct Buffer *s,
@@ -1269,11 +1269,11 @@ static int parse_unreplace_list(struct Buffer *buf, struct Buffer *s,
 
 static void clear_subject_mods(void)
 {
-  if (Context && Context->msgcount)
-  {
-    for (int i = 0; i < Context->msgcount; i++)
-      FREE(&Context->hdrs[i]->env->disp_subj);
-  }
+  if (!Context)
+    return;
+
+  for (int i = 0; i < Context->msgcount; i++)
+    FREE(&Context->hdrs[i]->env->disp_subj);
 }
 
 static int parse_subjectrx_list(struct Buffer *buf, struct Buffer *s,
@@ -1537,11 +1537,11 @@ bail:
  */
 static void attachments_clean(void)
 {
-  if (Context && Context->msgcount)
-  {
-    for (int i = 0; i < Context->msgcount; i++)
-      Context->hdrs[i]->attach_valid = false;
-  }
+  if (!Context)
+    return;
+
+  for (int i = 0; i < Context->msgcount; i++)
+    Context->hdrs[i]->attach_valid = false;
 }
 
 static int parse_attach_list(struct Buffer *buf, struct Buffer *s,