Re: [ovs-dev] [PATCH 27/41] ofp-print: Decode all async config messages the same way.

2016-01-20 Thread Ben Pfaff
Well, it unifies both. I changed it to "Decode and print". On Tue, Jan 19, 2016 at 04:23:46PM -0800, Jarno Rajahalme wrote: > Maybe the title should be “ofp-print: Print all async config messages the > same way.” instead? > > Acked-by: Jarno Rajahalme > > > On Jan 18, 2016, at 11:27 PM, Ben P

Re: [ovs-dev] [PATCH 27/41] ofp-print: Decode all async config messages the same way.

2016-01-19 Thread Jarno Rajahalme
Maybe the title should be “ofp-print: Print all async config messages the same way.” instead? Acked-by: Jarno Rajahalme > On Jan 18, 2016, at 11:27 PM, Ben Pfaff wrote: > > We have a single function to decode all of these messages, so there's no > reason to do it two different ways for printi

[ovs-dev] [PATCH 27/41] ofp-print: Decode all async config messages the same way.

2016-01-18 Thread Ben Pfaff
We have a single function to decode all of these messages, so there's no reason to do it two different ways for printing. Signed-off-by: Ben Pfaff --- lib/ofp-print.c | 117 +++--- tests/ofp-print.at| 26 +-- tests/ofproto-dpif.at |