Looks good. Ethan
On Fri, Sep 30, 2011 at 11:01, Ben Pfaff <b...@nicira.com> wrote: > This function was only pretty-printing "bad request" error payloads. I > don't know why. It makes sense to pretty-print all of them except for > "hello" messages, which already have their own special cases. > > Suggestion #7369. > Suggested-by: Reid Price <r...@nicira.com> > --- > lib/ofp-print.c | 6 +----- > 1 files changed, 1 insertions(+), 5 deletions(-) > > diff --git a/lib/ofp-print.c b/lib/ofp-print.c > index a9f90df..aa1a31f 100644 > --- a/lib/ofp-print.c > +++ b/lib/ofp-print.c > @@ -947,15 +947,11 @@ ofp_print_error_msg(struct ds *string, const struct > ofp_error_msg *oem) > ds_put_printable(string, payload, payload_len); > break; > > - case OFPET_BAD_REQUEST: > + default: > s = ofp_to_string(payload, payload_len, 1); > ds_put_cstr(string, s); > free(s); > break; > - > - default: > - ds_put_hex_dump(string, payload, payload_len, 0, true); > - break; > } > } > > -- > 1.7.4.4 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
