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>
ds_init(&s);
ds_put_hex_dump(&s, in, n_in * sizeof *a, 0, false);
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);
}
(n_in - left) * sizeof *a, ds_cstr(&s));
ds_destroy(&s);
}
if (!op->error && !ofproto_rule_is_hidden(rule)) {
/* Check that we can just cast from ofoperation_type to
* nx_flow_update_event. */
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,
ofmonitor_report(ofproto->connmgr, rule,
(enum nx_flow_update_event) op->type,