[ovs-dev] [cleanups 2/3] ofp-util: Remove obsolete build assertion.

Ethan Jackson ethan at nicira.com
Thu Sep 15 16:24:25 UTC 2011


looks good

On Thu, Sep 8, 2011 at 11:40, Ben Pfaff <blp at nicira.com> wrote:
> Commit d1e9b9bf3 "nicira-ext: Renumber NXT_FLOW_MOD_TABLE_ID" eliminated
> the need for the NXT_SET_FLOW_FORMAT and NXT_FLOW_MOD_TABLE_ID commands to
> have different sizes, so asserting that they are different isn't useful
> anymore (although it is still correct and always will be).
> ---
>  lib/ofp-util.c |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/lib/ofp-util.c b/lib/ofp-util.c
> index a4b1f5f..0a020d3 100644
> --- a/lib/ofp-util.c
> +++ b/lib/ofp-util.c
> @@ -353,9 +353,6 @@ static int
>  ofputil_decode_vendor(const struct ofp_header *oh,
>                       const struct ofputil_msg_type **typep)
>  {
> -    BUILD_ASSERT_DECL(sizeof(struct nxt_set_flow_format)
> -                      != sizeof(struct nxt_flow_mod_table_id));
> -
>     static const struct ofputil_msg_type nxt_messages[] = {
>         { OFPUTIL_NXT_ROLE_REQUEST,
>           NXT_ROLE_REQUEST, "NXT_ROLE_REQUEST",
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list