[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] vnc: simple clean up
From: |
Sahid Orentino Ferdjaoui |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] vnc: simple clean up |
Date: |
Fri, 7 Apr 2017 10:45:24 +0200 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Fri, Apr 07, 2017 at 02:58:58PM +0800, weiqi wrote:
> From: Wei Qi <address@hidden>
>
> It is unnecessary to assign 'packed_bytes' to 'estimated_bytes', because
> 'estimated_bytes' unused after assignment.
>
> Signed-off-by: Wei Qi <address@hidden>
> ---
> ui/vnc-enc-zrle.c | 1 -
> 1 file changed, 1 deletion(-)
Reviewed-By: Sahid Orentino Ferdjaoui <address@hidden>
>
> diff --git a/ui/vnc-enc-zrle.c b/ui/vnc-enc-zrle.c
> index 5489870..fd63d4f 100644
> --- a/ui/vnc-enc-zrle.c
> +++ b/ui/vnc-enc-zrle.c
> @@ -163,7 +163,6 @@ static void zrle_choose_palette_rle(VncState *vs, int w,
> int h,
> if (packed_bytes < estimated_bytes) {
> *use_rle = false;
> *use_palette = true;
> - estimated_bytes = packed_bytes;
I verified that 'estimated_bytes' is well a local variable defined in
'zrle_choose_palette_rle(...)'.
> }
> }
> }
> --
> 1.8.3.1
>
>