On Fri, Apr 07, 2017 at 02:58:58PM +0800, weiqi wrote: > From: Wei Qi <wei...@huawei.com> > > It is unnecessary to assign 'packed_bytes' to 'estimated_bytes', because > 'estimated_bytes' unused after assignment. > > Signed-off-by: Wei Qi <wei...@huawei.com> > --- > ui/vnc-enc-zrle.c | 1 - > 1 file changed, 1 deletion(-)
Reviewed-By: Sahid Orentino Ferdjaoui <sahid.ferdja...@redhat.com> > > 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 > >