Use temporary variable

-  Use the temporary variable more, as it helps readability: it removes
   a derefecence, which itself allows removing some parentheses.

-  Use a shorter name, which is more common with temporaries, and so
   there's less to read.

-  Assign to *ranges at the end of the function.  It's the same, but
   with the other changes, I think this makes it slightly clearer.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

1 files changed, 7 insertions, 6 deletions

diff --git a/lib/subordinateio.c b/lib/subordinateio.c
index 52d0e33d..4139674f 100644
--- a/lib/subordinateio.c
+++ b/lib/subordinateio.c
@@ -315,15 +315,16 @@ static bool have_range(struct commonio_db *db,
 
 static bool append_range(struct subid_range **ranges, const struct subordinate_range *new, int n)
 {
-	struct subid_range  *alloced;
+	struct subid_range  *sr;
 
-	alloced = REALLOC(*ranges, n + 1, struct subid_range);
-	if (!alloced)
+	sr = REALLOC(*ranges, n + 1, struct subid_range);
+	if (!sr)
 		return false;
-	*ranges = alloced;
 
-	(*ranges)[n].start = new->start;
-	(*ranges)[n].count = new->count;
+	sr[n].start = new->start;
+	sr[n].count = new->count;
+	*ranges = sr;
+
 	return true;
 }