smf: simplify get_child_of() loop and rename local to 'state'

Replace the open-ended while(true) with a null-terminated parent walk.
Improves readability, behavior is unchanged.

Signed-off-by: Vlad Kulikov <[email protected]>

Author: vlad-kulikov

lib/smf/smf.c

@@ -37,19 +37,17 @@ static bool is_descendant_of(const struct smf_state *test_state,
 static const struct smf_state *get_child_of(const struct smf_state *states,
 					    const struct smf_state *parent)
 {
-	const struct smf_state *tmp = states;
+	const struct smf_state *state = states;
 
-	while (true) {
-		if (tmp->parent == parent) {
-			return tmp;
+	while (state != NULL) {
+		if (state->parent == parent) {
+			return state;
 		}
 
-		if (tmp->parent == NULL) {
-			return NULL;
-		}
-
-		tmp = tmp->parent;
+		state = state->parent;
 	}
+
+	return NULL;
 }
 
 /**