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

Reply via email to