Re: [PATCH spice-gtk v3 6/6] usb-device-widget: Do not use GtkVBox

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

 



On Mon, 2017-01-23 at 14:54 +0100, Victor Toso wrote:
> Hi,
> 
> On Fri, Jan 20, 2017 at 04:53:53PM +0100, Pavel Grunt wrote:
> > ---
> >  src/usb-device-widget.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/usb-device-widget.c b/src/usb-device-widget.c
> > index b394499..747887c 100644
> > --- a/src/usb-device-widget.c
> > +++ b/src/usb-device-widget.c
> > @@ -40,14 +40,14 @@
> >  
> >  struct _SpiceUsbDeviceWidget
> >  {
> > -    GtkVBox parent;
> > +    GtkBox parent;
> >  
> >      SpiceUsbDeviceWidgetPrivate *priv;
> >  };
> >  
> >  struct _SpiceUsbDeviceWidgetClass
> >  {
> > -    GtkVBoxClass parent_class;
> > +    GtkBoxClass parent_class;
> 
> 
> "gtk_vbox_new has been deprecated since version 3.2 and should not
> be
>  used in newly-written code.
>  You can use gtk_box_new() with GTK_ORIENTATION_VERTICAL instead,
> which
>  is a quick and easy change. But the recommendation is to switch to
>  GtkGrid, since GtkBox is going to go away eventually. See Migrating
>  from other containers to GtkGrid."
> 
> So, GtkGrid in the future.
good point, no reason to do it in two step, we can switch to GtkGrid
right now.

Thanks,
Pavel

> Acked-by: Victor Toso <victortoso@xxxxxxxxxx>
> 
> >  
> >      /* signals */
> >      void (*connect_failed) (SpiceUsbDeviceWidget *widget,
> > -- 
> > 2.11.0
> > 
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




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