[virt-tools-list] [PATCH 1/3] virt-viewer-window: Store the monitor locally

[Christophe Fergeau]

On Thu, Mar 21, 2013 at 01:36:59PM +0100, Hans de Goede wrote:
> With commit 81ed9d13 "virt_viewer_window_enter_fullscreen: Pass in monitor for
> fullscreen window" we need a monitor number to determine where to move
> the window when going fullscreen.
> 
> Since the VirtViewerDisplay needs to know the fullscreen monitor number too,
> to determine the fullscreen size it was being stored there. But we don't
> always have a display, leading to errors like:
> 
> (remote-viewer:7996): remote-viewer-CRITICAL **:
> virt_viewer_display_get_monitor: assertion `VIRT_VIEWER_IS_DISPLAY(self)'
> failed
> 
> And to the monitor number not always being stored. This patchset fixes this
> by storing the monitor number inside VirtViewerWindow, and passing it to
> VirtViewerDisplay only when we've a display.

Patch by itself seems ok, however this makes me wonder if all of this is
not papering over that we are doing trying to do too much with
VirtViewerWindow too early (ie before we have a display)?

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20130321/31af6bbf/attachment.sig>