From c3e8f5c67376283d56e082c34f7c477867517dca Mon Sep 17 00:00:00 2001 From: Sebastian Harl Date: Sat, 22 Feb 2014 20:25:56 -0800 Subject: [PATCH] store_lookup_test: Continue loop after fail(). The compiler does not know that fail aborts the test program and clang (correctly) complains that the variable ('m' in this case) might be used uninitialized afterwards. --- t/core/store_lookup_test.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/t/core/store_lookup_test.c b/t/core/store_lookup_test.c index 48bb483..74274f8 100644 --- a/t/core/store_lookup_test.c +++ b/t/core/store_lookup_test.c @@ -293,8 +293,10 @@ START_TEST(test_store_match_op) else if (! strcmp(golden_data[i].op, "AND")) m = sdb_store_con_matcher(golden_data[i].left, golden_data[i].right); - else + else { fail("INTERNAL ERROR: unexpected operator %s", golden_data[i].op); + continue; + } #define TO_NAME(v) (((v) == always) ? "always" \ : ((v) == never) ? "never" : "") -- 2.30.2