On Fri, 2016-09-16 at 08:30 +0100, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx> > --- > server/dcc-send.c | 26 ++++++++++++-------------- > 1 file changed, 12 insertions(+), 14 deletions(-) > > diff --git a/server/dcc-send.c b/server/dcc-send.c > index 89c3dc5..1d32276 100644 > --- a/server/dcc-send.c > +++ b/server/dcc-send.c > @@ -102,28 +102,26 @@ static int > is_surface_area_lossy(DisplayChannelClient *dcc, uint32_t > surface_id, > if (!area) { > if (region_is_empty(surface_lossy_region)) { > return FALSE; > - } else { > - out_lossy_area->top = 0; > - out_lossy_area->left = 0; > - out_lossy_area->bottom = surface->context.height; > - out_lossy_area->right = surface->context.width; > - return TRUE; > } > + out_lossy_area->top = 0; > + out_lossy_area->left = 0; > + out_lossy_area->bottom = surface->context.height; > + out_lossy_area->right = surface->context.width; > + return TRUE; > } > > region_init(&lossy_region); > region_add(&lossy_region, area); > region_and(&lossy_region, surface_lossy_region); > - if (!region_is_empty(&lossy_region)) { > - out_lossy_area->left = lossy_region.extents.x1; > - out_lossy_area->top = lossy_region.extents.y1; > - out_lossy_area->right = lossy_region.extents.x2; > - out_lossy_area->bottom = lossy_region.extents.y2; > - region_destroy(&lossy_region); > - return TRUE; > - } else { > + if (region_is_empty(&lossy_region)) { > return FALSE; > } > + out_lossy_area->left = lossy_region.extents.x1; > + out_lossy_area->top = lossy_region.extents.y1; > + out_lossy_area->right = lossy_region.extents.x2; > + out_lossy_area->bottom = lossy_region.extents.y2; > + region_destroy(&lossy_region); > + return TRUE; > } > > /* returns if the bitmap was already sent lossy to the client. If > the bitmap hasn't been sent yet > _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel