[ovs-dev] [PATCH 1/2] ovs-brcompatd: Handle TXN_UNCHANGED status.

Ben Pfaff blp at nicira.com
Fri Dec 18 18:44:57 UTC 2009


This status was introduced several commits ago but not added to the switch
statement here.

Also, change the "status" variable to type enum ovsdb_idl_txn_status so
that GCC will warn about future additions.
---
 vswitchd/ovs-brcompatd.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/vswitchd/ovs-brcompatd.c b/vswitchd/ovs-brcompatd.c
index a446e7c..ad1c844 100644
--- a/vswitchd/ovs-brcompatd.c
+++ b/vswitchd/ovs-brcompatd.c
@@ -1183,7 +1183,7 @@ main(int argc, char *argv[])
     for (;;) {
         const struct ovsrec_open_vswitch *ovs;
         struct ovsdb_idl_txn *txn;
-        int status;
+        enum ovsdb_idl_txn_status status;
         unsigned int new_idl_seqno;
 
         ovsdb_idl_run(idl);
@@ -1249,6 +1249,7 @@ main(int argc, char *argv[])
             ovs_fatal(0, "transaction aborted");
         
         case TXN_SUCCESS:
+        case TXN_UNCHANGED:
             break;
         
         case TXN_TRY_AGAIN:
-- 
1.6.3.3





More information about the dev mailing list