From: Jani Nikula <jani@nikula.org>
Date: Fri, 6 Jan 2017 20:14:49 +0000 (+0200)
Subject: cli: simplify mime node walk
X-Git-Tag: 0.25_rc0~116
X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=commitdiff_plain;h=069362ee10d0af108c7d0a275bda1a78c4a49f27

cli: simplify mime node walk

The function is more straighforward to read when it's clear that the
only non-NULL return is at one place. No functional changes.
---

diff --git a/mime-node.c b/mime-node.c
index c9b82330..f719422e 100644
--- a/mime-node.c
+++ b/mime-node.c
@@ -322,20 +322,21 @@ mime_node_child (mime_node_t *parent, int child)
 static mime_node_t *
 _mime_node_seek_dfs_walk (mime_node_t *node, int *n)
 {
-    mime_node_t *ret = NULL;
     int i;
 
     if (*n == 0)
 	return node;
 
     *n -= 1;
-    for (i = 0; i < node->nchildren && !ret; i++) {
+    for (i = 0; i < node->nchildren; i++) {
 	mime_node_t *child = mime_node_child (node, i);
-	ret = _mime_node_seek_dfs_walk (child, n);
-	if (!ret)
-	    talloc_free (child);
+	mime_node_t *ret = _mime_node_seek_dfs_walk (child, n);
+	if (ret)
+	    return ret;
+
+	talloc_free (child);
     }
-    return ret;
+    return NULL;
 }
 
 mime_node_t *