Re: [ovs-dev] [PATCH 5/8] bundles: Reject unbundlable messages.

2014-12-22 Thread Jarno Rajahalme
On Dec 11, 2014, at 5:49 PM, Ben Pfaff wrote: > On Mon, Dec 08, 2014 at 02:01:31PM -0800, Jarno Rajahalme wrote: >> Reject bundle add messages containing messages that should not be bundled. >> >> Signed-off-by: Jarno Rajahalme > > Thanks for the patch! > > In ofputil_is_bundlable(), there's

Re: [ovs-dev] [PATCH 5/8] bundles: Reject unbundlable messages.

2014-12-11 Thread Ben Pfaff
On Mon, Dec 08, 2014 at 02:01:31PM -0800, Jarno Rajahalme wrote: > Reject bundle add messages containing messages that should not be bundled. > > Signed-off-by: Jarno Rajahalme Thanks for the patch! In ofputil_is_bundlable(), there's a "case" that falls through with an empty statement: > +c

[ovs-dev] [PATCH 5/8] bundles: Reject unbundlable messages.

2014-12-08 Thread Jarno Rajahalme
Reject bundle add messages containing messages that should not be bundled. Signed-off-by: Jarno Rajahalme --- lib/ofp-util.c | 94 tests/ofp-print.at | 47 ++ 2 files changed, 134 insertions(+), 7 deletions(-)