Zbigniew Jędrzejewski-Szmek 62fe94
From 080edb3484dc3ecf8d914526fdd3090b40fdf5b6 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Daniel Mack <daniel@zonque.org>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Fri, 19 Sep 2014 14:05:18 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] bus-policy: implement dump_items() with LIST_FOREACH
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Instead of making the function call itself recursively.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/bus-proxyd/bus-policy.c | 78 ++++++++++++++++++++++-----------------------
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 39 insertions(+), 39 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/bus-proxyd/bus-policy.c b/src/bus-proxyd/bus-policy.c
Zbigniew Jędrzejewski-Szmek 62fe94
index ab16cda32b..227742ba74 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/bus-proxyd/bus-policy.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/bus-proxyd/bus-policy.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -658,64 +658,64 @@ void policy_free(Policy *p) {
Zbigniew Jędrzejewski-Szmek 62fe94
         p->user_items = p->group_items = NULL;
Zbigniew Jędrzejewski-Szmek 62fe94
 }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-static void dump_items(PolicyItem *i, const char *prefix) {
Zbigniew Jędrzejewski-Szmek 62fe94
+static void dump_items(PolicyItem *items, const char *prefix) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (!i)
Zbigniew Jędrzejewski-Szmek 62fe94
+        PolicyItem *i;
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (!items)
Zbigniew Jędrzejewski-Szmek 62fe94
                 return;
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         if (!prefix)
Zbigniew Jędrzejewski-Szmek 62fe94
                 prefix = "";
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        printf("%sType: %s\n"
Zbigniew Jędrzejewski-Szmek 62fe94
-               "%sClass: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-               prefix, policy_item_type_to_string(i->type),
Zbigniew Jędrzejewski-Szmek 62fe94
-               prefix, policy_item_class_to_string(i->class));
Zbigniew Jędrzejewski-Szmek 62fe94
+        LIST_FOREACH(items, i, items) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->interface)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sInterface: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, i->interface);
Zbigniew Jędrzejewski-Szmek 62fe94
+                printf("%sType: %s\n"
Zbigniew Jędrzejewski-Szmek 62fe94
+                       "%sClass: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                       prefix, policy_item_type_to_string(i->type),
Zbigniew Jędrzejewski-Szmek 62fe94
+                       prefix, policy_item_class_to_string(i->class));
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->member)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sMember: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, i->member);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->interface)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sInterface: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, i->interface);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->error)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sError: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, i->error);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->member)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sMember: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, i->member);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->path)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sPath: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, i->path);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->error)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sError: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, i->error);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->name)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sName: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, i->name);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->path)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sPath: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, i->path);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->message_type != 0)
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sMessage Type: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, bus_message_type_to_string(i->message_type));
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->name)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sName: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, i->name);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->uid_valid) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                _cleanup_free_ char *user;
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->message_type != 0)
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sMessage Type: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, bus_message_type_to_string(i->message_type));
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-                user = uid_to_name(i->uid);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->uid_valid) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                        _cleanup_free_ char *user;
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sUser: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, strna(user));
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
+                        user = uid_to_name(i->uid);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->gid_valid) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                _cleanup_free_ char *group;
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sUser: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, strna(user));
Zbigniew Jędrzejewski-Szmek 62fe94
+                }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-                group = gid_to_name(i->gid);
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (i->gid_valid) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                        _cleanup_free_ char *group;
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%sGroup: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
-                       prefix, strna(group));
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
+                        group = gid_to_name(i->gid);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (i->items_next) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                printf("%s%s\n", prefix, draw_special_char(DRAW_DASH));
Zbigniew Jędrzejewski-Szmek 62fe94
-                dump_items(i->items_next, prefix);
Zbigniew Jędrzejewski-Szmek 62fe94
+                        printf("%sGroup: %s\n",
Zbigniew Jędrzejewski-Szmek 62fe94
+                               prefix, strna(group));
Zbigniew Jędrzejewski-Szmek 62fe94
+                }
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 }
Zbigniew Jędrzejewski-Szmek 62fe94