Ack series.
One remark about 2/4:
"spice: implement --fullscreen=auto-conf"
which is probably best fixed in a follow up patch.
Now that we have --fullscreen=auto-conf, we can
also implement the display-flags controller
CONTROLLER_AUTO_DISPLAY_RES flag properly, as
that is equivalent to --fullscreen=auto-conf,
so we should modify remote-viewer.c:
spice_ctrl_notified() to handle this and set
the "fullscreen-auto-conf" property on self
to the value of the CONTROLLER_AUTO_DISPLAY_RES.
Regards,
Hans
On 03/18/2012 07:59 PM, Marc-André Lureau wrote:
---
src/virt-viewer-app.c | 26 +++++++++++++-------------
1 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
index 789cb88..e30df85 100644
--- a/src/virt-viewer-app.c
+++ b/src/virt-viewer-app.c
@@ -680,20 +680,20 @@ virt_viewer_app_create_session(VirtViewerApp *self, const
gchar *type)
} else
#endif
#ifdef HAVE_SPICE_GTK
- if (g_ascii_strcasecmp(type, "spice") == 0) {
- GtkWindow *window =
virt_viewer_window_get_window(priv->main_window);
- virt_viewer_app_trace(self, "Guest %s has a %s display\n",
- priv->guest_name, type);
- priv->session = virt_viewer_session_spice_new(window);
- } else
+ if (g_ascii_strcasecmp(type, "spice") == 0) {
+ GtkWindow *window = virt_viewer_window_get_window(priv->main_window);
+ virt_viewer_app_trace(self, "Guest %s has a %s display\n",
+ priv->guest_name, type);
+ priv->session = virt_viewer_session_spice_new(window);
+ } else
#endif
- {
- virt_viewer_app_trace(self, "Guest %s has unsupported %s display
type\n",
- priv->guest_name, type);
- virt_viewer_app_simple_message_dialog(self, _("Unknown graphic type
for the guest %s"),
- priv->guest_name);
- return -1;
- }
+ {
+ virt_viewer_app_trace(self, "Guest %s has unsupported %s display
type\n",
+ priv->guest_name, type);
+ virt_viewer_app_simple_message_dialog(self, _("Unknown graphic type for the
guest %s"),
+ priv->guest_name);
+ return -1;
+ }
g_signal_connect(priv->session, "session-initialized",
G_CALLBACK(virt_viewer_app_initialized), self);
_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list
