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.

diff --git a/t/core/store_lookup_test.c b/t/core/store_lookup_test.c
index 48bb4832f4b81e723743d27cefd1c705e40e93a5..74274f80c0cf0f8fc6ec75b7a4dcc909c1c19449 100644 (file)
--- 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" : "<unknown>")