Re: [Qemu-devel] [PATCH 24/41] buffered_file: callers of buffered_flush() already check for errors

[Paolo Bonzini]

Il 21/09/2012 10:47, Juan Quintela ha scritto:
> Signed-off-by: Juan Quintela <address@hidden>
> ---
>  buffered_file.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/buffered_file.c b/buffered_file.c
> index 43e68b6..747d672 100644
> --- a/buffered_file.c
> +++ b/buffered_file.c
> @@ -61,13 +61,6 @@ static void buffered_append(QEMUFileBuffered *s,
>  static void buffered_flush(QEMUFileBuffered *s)
>  {
>      size_t offset = 0;
> -    int error;
> -
> -    error = qemu_file_get_error(s->file);
> -    if (error != 0) {
> -        DPRINTF("flush when error, bailing: %s\n", strerror(-error));
> -        return;
> -    }
> 
>      DPRINTF("flushing %zu byte(s) of data\n", s->buffer_size);
> 

Right, they do so _before_ checking buffered_flush which matches the
code you are removing here.

Perhaps an assertion would be better, but still:

Reviewed-by: Paolo Bonzini <address@hidden>