On Fri, Sep 05, 2014 at 04:05:11PM -0700, Jarno Rajahalme wrote: > Use the "+-" syntax more uniformly when printing masked flags, and use > the syntax of delimited 1-flags also for formatting fully masked TCP > flags. > > The "+-" syntax only deals with masked flags, but if there are many of > those, the printout becomes long and confusing. Typically there are > many flags only when flags are fully masked, but even then most of > them are zeros, so it makes sense to print the flags that are set > (ones) and omit the zero flags. > > Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>
I'd be inclined to just use "0" here: + } else { + ds_put_cstr(s, "0x000"); /* Zero flags. */ + } Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev