# Re: \[PATCH v3 09/11\] reftable: drop stray printf in readwrite\_test

- From: Ævar Arnfjörð Bjarmason \<hidden\>
- Date: 2021-12-13 16:27:02 
- List: git
- Page: https://ratatoskr.run/git/2021/12/7915643
- Thread: https://ratatoskr.run/git/2021/12/7914155/t.md

```text

On Mon, Dec 13 2021, Han-Wen Nienhuys via GitGitGadget wrote:

> From: Han-Wen Nienhuys <redacted>
>
> Signed-off-by: Han-Wen Nienhuys <redacted>
> ---
>  reftable/readwrite_test.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/reftable/readwrite_test.c b/reftable/readwrite_test.c
> index 42caf0bde4c..f6546d7b8d1 100644
> --- a/reftable/readwrite_test.c
> +++ b/reftable/readwrite_test.c
> @@ -663,7 +663,6 @@ static void test_write_key_order(void)
>  	err = reftable_writer_add_ref(w, &refs[0]);
>  	EXPECT_ERR(err);
>  	err = reftable_writer_add_ref(w, &refs[1]);
> -	printf("%d\n", err);
>  	EXPECT(err == REFTABLE_API_ERROR);
>  	reftable_writer_close(w);
>  	reftable_writer_free(w);

Is this something coverity raised and I'm missing why it's bad (per the
CL), or just a stray unrelated cleanup?
```