Re: [PATCH v2 11/16] session: do not take VTE display into account for geometry changes

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

 



Hi,

On Wed, Sep 26, 2018 at 07:26:34PM +0400, marcandre.lureau@xxxxxxxxxx wrote:
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  src/virt-viewer-session.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/src/virt-viewer-session.c b/src/virt-viewer-session.c
> index cd1f037..a40b0bb 100644
> --- a/src/virt-viewer-session.c
> +++ b/src/virt-viewer-session.c
> @@ -29,6 +29,7 @@
>  
>  #include "virt-viewer-session.h"
>  #include "virt-viewer-util.h"
> +#include "virt-viewer-display-vte.h"
>  
>  #define VIRT_VIEWER_SESSION_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE((o), VIRT_VIEWER_TYPE_SESSION, VirtViewerSessionPrivate))
>  
> @@ -417,6 +418,10 @@ virt_viewer_session_on_monitor_geometry_changed(VirtViewerSession* self,
>  
>      for (l = self->priv->displays; l; l = l->next) {
>          VirtViewerDisplay *d = VIRT_VIEWER_DISPLAY(l->data);
> +
> +        if (VIRT_VIEWER_IS_DISPLAY_VTE(d))
> +            continue;
> +

I was thinking about this while seeing 07/16, are both necessary?
Seems that this one is better than 06 and 07 as
virt_viewer_display_get_preferred_monitor_geometry is only called
here.

>          guint nth = 0;
>          GdkRectangle *rect = g_new0(GdkRectangle, 1);
>  
> -- 
> 2.19.0.271.gfe8321ec05
> 

Attachment: signature.asc
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux