Re: [spice-gtk PATCH] Handle single headed monitors that have a non-zero x, y config offset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 18, 2015 at 07:14:13PM -0400, sstutsma@xxxxxxxxxx wrote:
> From: Sandy Stutsman <sstutsma@xxxxxxxxxx>
> 
> Each monitor on a Windows guest is represented as a separate, single-headed
> device with its own framebuffer.  When there are multiple monitors, all
> monitors but one will have a non-zero xy config position.  But even in
> these cases the whole area (frame-buffer) of each monitor should be
> updated.
> 
> Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1202419
> ---
>  src/spice-widget.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/src/spice-widget.c b/src/spice-widget.c
> index 260fda5..f1e1508 100644
> --- a/src/spice-widget.c
> +++ b/src/spice-widget.c
> @@ -293,7 +293,12 @@ static void update_monitor_area(SpiceDisplay *display)
>          goto whole;
>      }
>  
> -    update_area(display, c->x, c->y, c->width, c->height);
> +    /* If only one head on this monitor, update the whole area */
> +    if(monitors->len == 1) {
> +        update_area(display, 0, 0, c->width, c->height);
> +    } else {
> +        update_area(display, c->x, c->y, c->width, c->height);
> +    }

What happens if c->x != 0 and c->y != 0 in the single monitor case and
we don't force these to 0?

Christophe

Attachment: pgp18FUJJ0P6B.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]