ofproto: Fix compiler warnings.

This patch fixes the following warnings on my system:

lib/ofp-actions.c:458:13: error: format '%x' expects argument of
type 'unsigned int', but argument 4 has type 'long unsigned int'
[-Werror=format]

ofproto/ofproto.c:3940:13: error: comparison between 'enum
ofoperation_type' and 'enum nx_flow_update_event'
[-Werror=enum-compare]

ofproto/ofproto.c:3941:13: error: comparison between 'enum
ofoperation_type' and 'enum nx_flow_update_event'
[-Werror=enum-compare]

ofproto/ofproto.c:3942:13: error: comparison between 'enum
ofoperation_type' and 'enum nx_flow_update_event'
[-Werror=enum-compare]

Signed-off-by: Ethan Jackson <ethan@nicira.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>

diff --git a/lib/ofp-actions.c b/lib/ofp-actions.c
index 582b5aea6f061de6ec8c565b484145cfe97d7e2e..5681fba8a94ff33d48339d721d99e263aef8684f 100644 (file)
--- a/lib/ofp-actions.c
+++ b/lib/ofp-actions.c
@@ -455,7 +455,7 @@ ofpacts_from_openflow10(const union ofp_action *in, size_t n_in,
 
             ds_init(&s);
             ds_put_hex_dump(&s, in, n_in * sizeof *a, 0, false);
-            VLOG_WARN("bad action format at offset %#x:\n%s",
+            VLOG_WARN("bad action format at offset %#zx:\n%s",
                       (n_in - left) * sizeof *a, ds_cstr(&s));
             ds_destroy(&s);
         }

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index b187c86fbc3c7fb89e3d6123bbddd481bc7bd859..4daa0cd77e96f9a20b09e8f255047a891895c6d5 100644 (file)
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -3937,9 +3937,12 @@ ofopgroup_complete(struct ofopgroup *group)
         if (!op->error && !ofproto_rule_is_hidden(rule)) {
             /* Check that we can just cast from ofoperation_type to
              * nx_flow_update_event. */
-            BUILD_ASSERT_DECL(OFOPERATION_ADD == NXFME_ADDED);
-            BUILD_ASSERT_DECL(OFOPERATION_DELETE == NXFME_DELETED);
-            BUILD_ASSERT_DECL(OFOPERATION_MODIFY == NXFME_MODIFIED);
+            BUILD_ASSERT_DECL((enum nx_flow_update_event) OFOPERATION_ADD
+                              == NXFME_ADDED);
+            BUILD_ASSERT_DECL((enum nx_flow_update_event) OFOPERATION_DELETE
+                              == NXFME_DELETED);
+            BUILD_ASSERT_DECL((enum nx_flow_update_event) OFOPERATION_MODIFY
+                              == NXFME_MODIFIED);
 
             ofmonitor_report(ofproto->connmgr, rule,
                              (enum nx_flow_update_event) op->type,