----- Original Message -----
> ACK, but a general comment/question:
>
> At the moment, the menu object is kind of floating and not owned by anybody.
> This means that it is leaked, which isn't a big deal since it's just a
> single object that lives the whole life of the application. But it also
> means there's no way to reference it outside of a signal handler. It might
> be cleaner to have it owned by the app object? But maybe that will create
> too much of an ifdef mess...
>
>
well, if that's only the leak that worries you, you could use g_object_set_data_full(app, ..) ?
> ----- Original Message -----
> > From: "Christophe Fergeau" <cfergeau@xxxxxxxxxx>
> > To: virt-tools-list@xxxxxxxxxx
> > Sent: Wednesday, April 16, 2014 11:59:54 AM
> > Subject: [virt-viewer 7/7] Create foreign menu from .vv
> > file information
> >
> > When the .vv file has an [ovirt] section, we should try to create a foreign
> > menu out of it. This will allow remote-viewer to offer a menu to change the
> > currenty inserted cdrom.
> > ---
> > src/virt-viewer-file.c | 15 +++++++++++++++
> > 1 file changed, 15 insertions(+)
> >
> > diff --git a/src/virt-viewer-file.c b/src/virt-viewer-file.c
> > index 640b664..396df93 100644
> > --- a/src/virt-viewer-file.c
> > +++ b/src/virt-viewer-file.c
> > @@ -24,6 +24,10 @@
> > #include <glib/gi18n.h>
> > #include <glib/gstdio.h>
> >
> > +#ifdef HAVE_OVIRT
> > +#include "ovirt-foreign-menu.h"
> > +#endif
> > +
> > #include "virt-viewer-util.h"
> > #include "virt-viewer-file.h"
> >
> > @@ -749,6 +753,17 @@ virt_viewer_file_fill_app(VirtViewerFile* self,
> > VirtViewerApp *app, GError **err
> > g_object_set(G_OBJECT(app), "fullscreen",
> > virt_viewer_file_get_fullscreen(self), NULL);
> >
> > +#ifdef HAVE_OVIRT
> > + {
> > + OvirtForeignMenu *menu;
> > +
> > + if (g_key_file_has_group(self->priv->keyfile, OVIRT_GROUP)) {
> > + menu = ovirt_foreign_menu_new_from_file(self);
> > + ovirt_foreign_menu_start(menu);
> > + }
> > + }
> > +#endif
> > +
> > return TRUE;
> > }
> >
> > --
> > 1.9.0
> >
> > _______________________________________________
> > virt-tools-list mailing list
> > virt-tools-list@xxxxxxxxxx
> > https://www.redhat.com/mailman/listinfo/virt-tools-list
> >
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/virt-tools-list
>
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
