On Thu, Nov 15, 2012 at 10:44:19PM -0800, Ben Pfaff wrote: > See http://openvswitch.org/pipermail/dev/2012-November/022990.html > for background. > > Ben Pfaff (5): > ovs-ofctl: Style fix. > ofp-util: Rename OFPUTIL_P_* constants and update comments for > clarity. > ofp-util: Allow use of OpenFlow 12 flow format > ofp-util: Make ofputil_encode_set_protocol() able to return failure. > nicira-ext: Remove NXFF_OPENFLOW12. > > include/openflow/nicira-ext.h | 3 +- > lib/learning-switch.c | 20 +++-- > lib/ofp-msgs.h | 2 +- > lib/ofp-util.c | 191 > +++++++++++++++++++++-------------------- > lib/ofp-util.h | 52 +++++++---- > ofproto/connmgr.c | 4 +- > tests/learn.at | 2 +- > tests/ovs-ofctl.at | 12 +-- > utilities/ovs-ofctl.c | 5 +- > 9 files changed, 159 insertions(+), 132 deletions(-)
This all seems reasonable to me. Reviewed-by: Simon Horman <ho...@verge.net.au> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev