Re: [Qemu-devel] [PATCH] qemu-ga: drop unused local err variable

[Michael Roth <mdroth@linux.vnet.ibm.com>]

Quoting Stefan Hajnoczi (2016-03-16 10:16:02)
> Commit 125b310e1d62e3a1dc1e7758563e598957ca7ae4 ("qemu-ga: move
> channel/transport functionality into wrapper class") stopped using the
> local err variable in channel_event_cb().
> 
> This patch deletes the unused variable.
> 
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

Thanks, applied to qga tree:

  https://github.com/mdroth/qemu/commits/qga

> ---
>  qga/main.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/qga/main.c b/qga/main.c
> index 0a168e2..743c5c1 100644
> --- a/qga/main.c
> +++ b/qga/main.c
> @@ -618,13 +618,7 @@ static gboolean channel_event_cb(GIOCondition condition, gpointer data)
>      GAState *s = data;
>      gchar buf[QGA_READ_COUNT_DEFAULT+1];
>      gsize count;
> -    GError *err = NULL;
>      GIOStatus status = ga_channel_read(s->channel, buf, QGA_READ_COUNT_DEFAULT, &count);
> -    if (err != NULL) {
> -        g_warning("error reading channel: %s", err->message);
> -        g_error_free(err);
> -        return false;
> -    }
>      switch (status) {
>      case G_IO_STATUS_ERROR:
>          g_warning("error reading channel");
> -- 
> 2.5.0
>