[ovs-dev] [PATCH ovn v2] ofctrl: Don't use designated initializers.
Gurucharan Shetty
shettyg at nicira.com
Tue Jun 16 13:44:35 UTC 2015
MSVC 2013 does not like designated initializers when
structs are initialized inside structs.
Apparently it has been fixed in MSVC 2015.
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
---
ovn/controller/ofctrl.c | 44 ++++++++++++++++++++++----------------------
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/ovn/controller/ofctrl.c b/ovn/controller/ofctrl.c
index bc4e8d4..2f1767a 100644
--- a/ovn/controller/ofctrl.c
+++ b/ovn/controller/ofctrl.c
@@ -401,12 +401,12 @@ ofctrl_update_flows(void)
if (!d) {
/* Installed flow is no longer desirable. Delete it from the
* switch and from installed_flows. */
- struct ofputil_flow_mod fm = {
- .match = i->match,
- .priority = i->priority,
- .table_id = i->table_id,
- .command = OFPFC_DELETE_STRICT,
- };
+ struct ofputil_flow_mod fm;
+ memset(&fm, 0, sizeof fm);
+ fm.match = i->match;
+ fm.priority = i->priority;
+ fm.table_id = i->table_id;
+ fm.command = OFPFC_DELETE_STRICT;
queue_flow_mod(&fm);
ovn_flow_log(i, "removing");
@@ -416,14 +416,14 @@ ofctrl_update_flows(void)
if (!ofpacts_equal(i->ofpacts, i->ofpacts_len,
d->ofpacts, d->ofpacts_len)) {
/* Update actions in installed flow. */
- struct ofputil_flow_mod fm = {
- .match = i->match,
- .priority = i->priority,
- .table_id = i->table_id,
- .ofpacts = d->ofpacts,
- .ofpacts_len = d->ofpacts_len,
- .command = OFPFC_MODIFY_STRICT,
- };
+ struct ofputil_flow_mod fm;
+ memset(&fm, 0, sizeof fm);
+ fm.match = i->match;
+ fm.priority = i->priority;
+ fm.table_id = i->table_id;
+ fm.ofpacts = d->ofpacts;
+ fm.ofpacts_len = d->ofpacts_len;
+ fm.command = OFPFC_MODIFY_STRICT;
queue_flow_mod(&fm);
ovn_flow_log(i, "updating");
@@ -446,14 +446,14 @@ ofctrl_update_flows(void)
struct ovn_flow *d;
HMAP_FOR_EACH_SAFE (d, next, hmap_node, &desired_flows) {
/* Send flow_mod to add flow. */
- struct ofputil_flow_mod fm = {
- .match = d->match,
- .priority = d->priority,
- .table_id = d->table_id,
- .ofpacts = d->ofpacts,
- .ofpacts_len = d->ofpacts_len,
- .command = OFPFC_ADD,
- };
+ struct ofputil_flow_mod fm;
+ memset(&fm, 0, sizeof fm);
+ fm.match = d->match;
+ fm.priority = d->priority;
+ fm.table_id = d->table_id;
+ fm.ofpacts = d->ofpacts;
+ fm.ofpacts_len = d->ofpacts_len;
+ fm.command = OFPFC_ADD;
queue_flow_mod(&fm);
ovn_flow_log(d, "adding");
--
1.7.9.5
More information about the dev
mailing list