On Sun, Jul 17, 2016 at 11:13:10PM -0300, Eduardo Lima (Etrunko) wrote: > The OvirtForeignMenu pointer is needed by the new ISO list dialog, and > we make it acessible via property to avoid interdependency between > objects. > > Signed-off-by: Eduardo Lima (Etrunko) <[email protected]> > --- > src/remote-viewer-iso-list-dialog.c | 20 +++++++++++++++----- > src/remote-viewer-iso-list-dialog.h | 3 ++- > src/remote-viewer.c | 36 ++++++++++++++++++++++++++++++++++++ > src/virt-viewer-window.c | 16 +++++++++++++++- > 4 files changed, 68 insertions(+), 7 deletions(-) > > diff --git a/src/remote-viewer-iso-list-dialog.c > b/src/remote-viewer-iso-list-dialog.c > index b3972ac..996c1f2 100644 > --- a/src/remote-viewer-iso-list-dialog.c > +++ b/src/remote-viewer-iso-list-dialog.c > @@ -34,6 +34,7 @@ struct _RemoteViewerISOListDialogPrivate > { > GtkBuilder *builder; > GtkWidget *notebook; > + OvirtForeignMenu *foreign_menu; > }; > > enum RemoteViewerIsoListDialogPages > @@ -106,10 +107,19 @@ > remote_viewer_iso_list_dialog_init(RemoteViewerISOListDialog *self) > } > > GtkWidget * > -remote_viewer_iso_list_dialog_new(GtkWindow *parent) > +remote_viewer_iso_list_dialog_new(GtkWindow *parent, OvirtForeignMenu > *foreign_menu) > { > - return g_object_new(REMOTE_VIEWER_TYPE_ISO_LIST_DIALOG, > - "title", _("Change CD"), > - "transient-for", parent, > - NULL); > + GtkWidget *dialog; > + RemoteViewerISOListDialog *self; > + > + g_return_val_if_fail(foreign_menu != NULL, NULL); > + > + dialog = g_object_new(REMOTE_VIEWER_TYPE_ISO_LIST_DIALOG, > + "title", _("Change CD"), > + "transient-for", parent, > + NULL); > + > + self = REMOTE_VIEWER_ISO_LIST_DIALOG(dialog); > + self->priv->foreign_menu = foreign_menu;
I'd g_object_ref it if you need to have it around (together with g_clear_object
in dispose/finalize).
> + return dialog;
> }
> diff --git a/src/remote-viewer-iso-list-dialog.h
> b/src/remote-viewer-iso-list-dialog.h
> index def841b..ee7c70f 100644
> --- a/src/remote-viewer-iso-list-dialog.h
> +++ b/src/remote-viewer-iso-list-dialog.h
> @@ -22,6 +22,7 @@
> #define __REMOTE_VIEWER_ISO_LIST_DIALOG_H__
>
> #include <gtk/gtk.h>
> +#include "ovirt-foreign-menu.h"
>
> G_BEGIN_DECLS
>
> @@ -51,7 +52,7 @@ struct _RemoteViewerISOListDialogClass
>
> GType remote_viewer_iso_list_dialog_get_type(void) G_GNUC_CONST;
>
> -GtkWidget *remote_viewer_iso_list_dialog_new(GtkWindow *parent);
> +GtkWidget *remote_viewer_iso_list_dialog_new(GtkWindow *parent,
> OvirtForeignMenu *foreign_menu);
>
> G_END_DECLS
>
> diff --git a/src/remote-viewer.c b/src/remote-viewer.c
> index 95130dc..e4e41c4 100644
> --- a/src/remote-viewer.c
> +++ b/src/remote-viewer.c
> @@ -67,6 +67,13 @@ G_DEFINE_TYPE (RemoteViewer, remote_viewer,
> VIRT_VIEWER_TYPE_APP)
> #define GET_PRIVATE(o)
> \
> (G_TYPE_INSTANCE_GET_PRIVATE ((o), REMOTE_VIEWER_TYPE,
> RemoteViewerPrivate))
>
> +enum RemoteViewerProperties {
> + PROP_0,
> +#ifdef HAVE_OVIRT
> + PROP_OVIRT_FOREIGN_MENU,
> +#endif
> +};
> +
> #ifdef HAVE_OVIRT
> static OvirtVm * choose_vm(GtkWindow *main_window,
> char **vm_name,
> @@ -213,6 +220,25 @@ end:
> }
>
> static void
> +remote_viewer_get_property(GObject *object, guint property_id,
> + GValue *value, GParamSpec *pspec)
> +{
> + RemoteViewer *self = REMOTE_VIEWER(object);
> + RemoteViewerPrivate *priv = self->priv;
> +
> + switch (property_id) {
> +#ifdef HAVE_OVIRT
> + case PROP_OVIRT_FOREIGN_MENU:
> + g_value_set_pointer(value, priv->ovirt_foreign_menu);
> + break;
> +#endif
> +
> + default:
> + G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
> + }
> +}
> +
> +static void
> remote_viewer_class_init (RemoteViewerClass *klass)
> {
> GObjectClass *object_class = G_OBJECT_CLASS (klass);
> @@ -222,6 +248,7 @@ remote_viewer_class_init (RemoteViewerClass *klass)
>
> g_type_class_add_private (klass, sizeof (RemoteViewerPrivate));
>
> + object_class->get_property = remote_viewer_get_property;
> object_class->dispose = remote_viewer_dispose;
>
> g_app_class->local_command_line = remote_viewer_local_command_line;
> @@ -235,6 +262,15 @@ remote_viewer_class_init (RemoteViewerClass *klass)
> #else
> (void) gtk_app_class;
> #endif
> +
> +#ifdef HAVE_OVIRT
> + g_object_class_install_property(object_class,
> + PROP_OVIRT_FOREIGN_MENU,
> +
> g_param_spec_pointer("ovirt-foreign-menu",
This can be a g_param_spec_object, OvirtForeignMenu is a GObject.
> + "oVirt Foreign
> Menu",
> + "Object which is
> used as interface to oVirt",
> + G_PARAM_READABLE |
> G_PARAM_STATIC_STRINGS));
> +#endif
> }
>
> static void
> diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
> index 867fb86..76fe80f 100644
> --- a/src/virt-viewer-window.c
> +++ b/src/virt-viewer-window.c
> @@ -1072,11 +1072,25 @@ virt_viewer_window_menu_change_cd_activate(GtkWidget
> *menu G_GNUC_UNUSED,
> {
> VirtViewerWindowPrivate *priv = self->priv;
> static GtkWidget *dialog = NULL;
> + GValue foreign_menu = G_VALUE_INIT;
>
> if (dialog)
> return;
>
> - dialog = remote_viewer_iso_list_dialog_new(GTK_WINDOW(priv->window));
> + g_value_init(&foreign_menu, G_TYPE_POINTER);
> + g_object_get_property(G_OBJECT(priv->app), "ovirt-foreign-menu",
> &foreign_menu);
You can use g_object_get(G_OBJECT(priv->app), "ovirt-foreign_menu",
&foreign_menu, NULL);
rather than a GValue.
Christophe
signature.asc
Description: PGP signature
_______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
