[ovs-dev] [minor cleanups 3/5] Avoid writes to variables that are never read back.

Ben Pfaff blp at nicira.com
Sun May 13 23:58:57 UTC 2012


Found by clang.

Signed-off-by: Ben Pfaff <blp at nicira.com>
---
 lib/ofp-print.c       |    2 +-
 utilities/ovs-vsctl.c |    2 --
 2 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/lib/ofp-print.c b/lib/ofp-print.c
index 7479bf2..1757a30 100644
--- a/lib/ofp-print.c
+++ b/lib/ofp-print.c
@@ -480,7 +480,7 @@ ofp_print_bit_names(struct ds *string, uint32_t bits,
     }
 
     if (bits) {
-        if (n++) {
+        if (n) {
             ds_put_char(string, ' ');
         }
         ds_put_format(string, "0x%"PRIx32, bits);
diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
index 2fc87f0..d8aeb53 100644
--- a/utilities/ovs-vsctl.c
+++ b/utilities/ovs-vsctl.c
@@ -3809,8 +3809,6 @@ do_vsctl(const char *args, struct vsctl_command *commands, size_t n_commands,
 
         if (ctx.try_again) {
             vsctl_context_done(&ctx, NULL);
-
-            status = TXN_TRY_AGAIN;
             goto try_again;
         }
     }
-- 
1.7.2.5




More information about the dev mailing list