Re: [PATCH spice-gtk 2/3] usb-device-widget: replace deprecated Gtk+ API usage

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

 



ping

On Sat, Dec 22, 2018 at 6:53 PM <marcandre.lureau@xxxxxxxxxx> wrote:
>
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
>
> Use widget properties instead, as recommended in the documentation.
> The UI doesn't seem to have changed after these modifications.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  src/usb-device-widget.c | 23 +++++++++--------------
>  1 file changed, 9 insertions(+), 14 deletions(-)
>
> diff --git a/src/usb-device-widget.c b/src/usb-device-widget.c
> index da24d00..de62564 100644
> --- a/src/usb-device-widget.c
> +++ b/src/usb-device-widget.c
> @@ -163,9 +163,9 @@ spice_usb_device_widget_show_info_bar(SpiceUsbDeviceWidget *self,
>      widget = gtk_label_new(message);
>      gtk_box_pack_start(GTK_BOX(hbox), widget, TRUE, TRUE, 0);
>
> -    priv->info_bar = gtk_alignment_new(0.0, 0.0, 1.0, 0.0);
> -    gtk_alignment_set_padding(GTK_ALIGNMENT(priv->info_bar), 0, 0, 12, 0);
> -    gtk_container_add(GTK_CONTAINER(priv->info_bar), info_bar);
> +    priv->info_bar = info_bar;
> +    gtk_widget_set_margin_start(info_bar, 12);
> +    gtk_widget_set_halign(info_bar, GTK_ALIGN_FILL);
>      gtk_box_pack_start(GTK_BOX(self), priv->info_bar, FALSE, FALSE, 0);
>      gtk_widget_show_all(priv->info_bar);
>  }
> @@ -187,7 +187,8 @@ static void spice_usb_device_widget_constructed(GObject *gobject)
>      str = g_strdup_printf("<b>%s</b>", _("Select USB devices to redirect"));
>      gtk_label_set_markup(GTK_LABEL (priv->label), str);
>      g_free(str);
> -    gtk_misc_set_alignment(GTK_MISC(priv->label), 0.0, 0.5);
> +    gtk_label_set_xalign(GTK_LABEL(priv->label), 0.0);
> +    gtk_label_set_yalign(GTK_LABEL(priv->label), 0.5);
>      gtk_box_pack_start(GTK_BOX(self), priv->label, FALSE, FALSE, 0);
>
>      priv->manager = spice_usb_device_manager_get(priv->session, &err);
> @@ -336,10 +337,6 @@ GtkWidget *spice_usb_device_widget_new(SpiceSession    *session,
>
>  static SpiceUsbDevice *get_usb_device(GtkWidget *widget)
>  {
> -    if (!GTK_IS_ALIGNMENT(widget))
> -        return NULL;
> -
> -    widget = gtk_bin_get_child(GTK_BIN(widget));
>      return g_object_get_data(G_OBJECT(widget), "usb-device");
>  }
>
> @@ -525,7 +522,7 @@ static void device_added_cb(SpiceUsbDeviceManager *manager,
>  {
>      SpiceUsbDeviceWidget *self = SPICE_USB_DEVICE_WIDGET(user_data);
>      SpiceUsbDeviceWidgetPrivate *priv = self->priv;
> -    GtkWidget *align, *check;
> +    GtkWidget *check;
>      gchar *desc;
>
>      desc = spice_usb_device_get_description(device,
> @@ -544,12 +541,10 @@ static void device_added_cb(SpiceUsbDeviceManager *manager,
>      g_signal_connect(G_OBJECT(check), "clicked",
>                       G_CALLBACK(checkbox_clicked_cb), self);
>
> -    align = gtk_alignment_new(0, 0, 0, 0);
> -    gtk_alignment_set_padding(GTK_ALIGNMENT(align), 0, 0, 12, 0);
> -    gtk_container_add(GTK_CONTAINER(align), check);
> -    gtk_box_pack_end(GTK_BOX(self), align, FALSE, FALSE, 0);
> +    gtk_widget_set_margin_start(check, 12);
> +    gtk_box_pack_end(GTK_BOX(self), check, FALSE, FALSE, 0);
>      spice_usb_device_widget_update_status(self);
> -    gtk_widget_show_all(align);
> +    gtk_widget_show_all(check);
>  }
>
>  static void destroy_widget_by_usb_device(GtkWidget *widget, gpointer user_data)
> --
> 2.20.1.2.gb21ebb671b
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]