On Mon, Jul 8, 2013 at 10:13 AM, Ben Pfaff <[email protected]> wrote: > > 1. Could you explain why the last case "OVSINST_OFPIT11_GOTO_TABLE" is > > enclosed by curly bracket? > > It has a local variable, so the braces are necessary.
Actually, I didn't know it. Good to learn. On Mon, Jul 8, 2013 at 10:13 AM, Ben Pfaff <[email protected]> wrote: > > Actually, I'm surprised by how few tests are modified. I'll do a scan > > again, after this patch is updated. > > This indicates that we have poor coverage of "negative" tests (the > ones that provoke error messages). It would be good to fix that. I think I can try adding more negative tests.
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
