We've forgotten more than once. Most updates coincide with adding new flow members, so a build assertion on FLOW_WC_SEQ seems appropriate.
Signed-off-by: Ben Pfaff <b...@nicira.com> --- lib/odp-util.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/odp-util.h b/lib/odp-util.h index 58bbce9..d06c385 100644 --- a/lib/odp-util.h +++ b/lib/odp-util.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2009, 2010, 2011, 2012, 2013 Nicira, Inc. + * Copyright (c) 2009, 2010, 2011, 2012, 2013, 2014 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -22,15 +22,13 @@ #include <stdint.h> #include <string.h> #include <linux/openvswitch.h> +#include "flow.h" #include "hash.h" #include "hmap.h" #include "openflow/openflow.h" #include "util.h" struct ds; -struct flow; -struct flow_tnl; -struct flow_wildcards; struct nlattr; struct ofpbuf; struct simap; @@ -123,6 +121,7 @@ void odp_portno_names_destroy(struct hmap *portno_names); * add another field and forget to adjust this value. */ #define ODPUTIL_FLOW_KEY_BYTES 256 +BUILD_ASSERT_DECL(FLOW_WC_SEQ == 26); /* A buffer with sufficient size and alignment to hold an nlattr-formatted flow * key. An array of "struct nlattr" might not, in theory, be sufficiently -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
