Signed-off-by: Ben Pfaff <[email protected]>
---
 ovn/utilities/ovn-trace.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c
index b1ffe1f..b5ccc28 100644
--- a/ovn/utilities/ovn-trace.c
+++ b/ovn/utilities/ovn-trace.c
@@ -823,13 +823,17 @@ ovntrace_stage_name(const struct ovntrace_datapath *dp,
     return NULL;
 }
 
+/* Type of a node within a trace. */
 enum ovntrace_node_type {
+    /* Nodes that may have children (nonterminal nodes). */
     OVNTRACE_NODE_OUTPUT,
     OVNTRACE_NODE_MODIFY,
-    OVNTRACE_NODE_PIPELINE,
-    OVNTRACE_NODE_TABLE,
     OVNTRACE_NODE_ACTION,
     OVNTRACE_NODE_ERROR,
+
+    /* Nodes that never have children (terminal nodes). */
+    OVNTRACE_NODE_PIPELINE,
+    OVNTRACE_NODE_TABLE,
     OVNTRACE_NODE_TRANSFORMATION
 };
 
@@ -927,7 +931,9 @@ ovntrace_node_print_details(struct ds *output,
             continue;
         }
 
-        bool more = sub->node.next != nodes || sub->always_indent || 
ovntrace_node_type_is_terminal(sub->type);
+        bool more = (sub->node.next != nodes
+                     || sub->always_indent
+                     || ovntrace_node_type_is_terminal(sub->type));
         bool title = (sub->type == OVNTRACE_NODE_PIPELINE ||
                       sub->type == OVNTRACE_NODE_TRANSFORMATION);
         if (title) {
-- 
2.10.2

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to