[ovs-dev] [PATCH] ofp-print: Merge two cases in ofp_to_string__().

Ben Pfaff blp at nicira.com
Wed Feb 8 19:24:00 UTC 2012


On Tue, Feb 07, 2012 at 04:52:44PM -0800, Justin Pettit wrote:
> On Feb 7, 2012, at 2:58 PM, Ben Pfaff wrote:
> 
> > @@ -1524,8 +1525,6 @@ ofp_to_string__(const struct ofp_header *oh,
> >         ofp_print_nxt_set_packet_in_format(string, msg);
> >         break;
> > 
> > -    case OFPUTIL_NXT_FLOW_MOD:
> > -        ofp_print_flow_mod(string, msg, code, verbosity);
> >         break;
> 
> I think you've got an extra "break" in there now.

That was a dumb mistake.  Thanks, I pushed a fix.



More information about the dev mailing list