On Wed 09 Dec 2020 05:44:39 PM CET, Maxim Levitsky wrote: > +void coroutine_fn bdrv_co_delete_file_noerr(BlockDriverState *bs) > +{ > + Error *local_err = NULL; > + > + if (!bs) { > + return; > + } > + > + int ret = bdrv_co_delete_file(bs, &local_err); ^^^
According to the QEMU coding style we should not have declarations in the middle of a block. The patch looks otherwise fine. Reviewed-by: Alberto Garcia <be...@igalia.com> Berto