I will go ahead and change it to ds_put_char before merging. Ethan
On Mon, Feb 14, 2011 at 3:28 PM, Ben Pfaff <[email protected]> wrote: > On Mon, Feb 14, 2011 at 02:05:58PM -0800, Ethan Jackson wrote: >> @@ -3668,6 +3668,7 @@ flow_stats_ds(struct ofproto *ofproto, struct rule >> *rule, struct ds *results) >> ds_put_format(results, "n_packets=%"PRIu64", ", packet_count); >> ds_put_format(results, "n_bytes=%"PRIu64", ", byte_count); >> cls_rule_format(&rule->cr, results); >> + ds_put_format(results, ","); >> if (act_len > 0) { >> ofp_print_actions(results, &rule->actions->header, act_len); >> } else { > > Looks fine. Since it's a single character it would also be reasonable > to use ds_put_char(). > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
