On Thu, Jan 20 2022, Han-Wen Nienhuys via GitGitGadget wrote: > From: Han-Wen Nienhuys <hanwen@xxxxxxxxxx> > > Fixes a problem detected by Coverity. Detected by coverity, but I think it's better to describe the issue. Here it seems to be that we didn't use the "err" at all, so before this change the test wasn't doing anything to check that we didn't have an error? Or would the subsequent EXPECT() catch it in practice? > Signed-off-by: Han-Wen Nienhuys <hanwen@xxxxxxxxxx> > --- > reftable/stack_test.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/reftable/stack_test.c b/reftable/stack_test.c > index f4c743db80c..e84f50d27ff 100644 > --- a/reftable/stack_test.c > +++ b/reftable/stack_test.c > @@ -839,6 +839,7 @@ static void test_reftable_stack_auto_compaction(void) > EXPECT_ERR(err); > > err = reftable_stack_auto_compact(st); > + EXPECT_ERR(err); > EXPECT(i < 3 || st->merged->stack_len < 2 * fastlog2(i)); > }
