From a893b20eed778cb5928f6412baed249a9909f11d Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Thu, 4 Sep 2014 15:30:33 -0400 Subject: [PATCH] tests: Resolve Coverity DEADCODE Coverity complains that the various checks for autoincrement and changed variables are DEADCODE - seems to me to be a false positive - so mark it. Signed-off-by: John Ferlan --- tests/virstringtest.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/virstringtest.c b/tests/virstringtest.c index 7c25b22c67..10fad2c880 100644 --- a/tests/virstringtest.c +++ b/tests/virstringtest.c @@ -162,10 +162,12 @@ testStrdup(const void *data ATTRIBUTE_UNUSED) virFilePrintf(stderr, "unexpected strdup result %d, expected 1\n", value); goto cleanup; } + /* coverity[dead_error_begin] */ if (i != 1) { virFilePrintf(stderr, "unexpected side effects i=%zu, expected 1\n", i); goto cleanup; } + /* coverity[dead_error_begin] */ if (j != 1) { virFilePrintf(stderr, "unexpected side effects j=%zu, expected 1\n", j); goto cleanup; @@ -182,14 +184,17 @@ testStrdup(const void *data ATTRIBUTE_UNUSED) virFilePrintf(stderr, "unexpected strdup result %d, expected 0\n", value); goto cleanup; } + /* coverity[dead_error_begin] */ if (i != 2) { virFilePrintf(stderr, "unexpected side effects i=%zu, expected 2\n", i); goto cleanup; } + /* coverity[dead_error_begin] */ if (j != 2) { virFilePrintf(stderr, "unexpected side effects j=%zu, expected 2\n", j); goto cleanup; } + /* coverity[dead_error_begin] */ if (k != 1) { virFilePrintf(stderr, "unexpected side effects k=%zu, expected 1\n", k); goto cleanup;