Re: [virt-viewer] session-vnc: Avoid depracated warnings on this file

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

 



On Fri, 2016-02-26 at 18:50 +0100, Fabiano Fidêncio wrote:
> On Fri, Feb 26, 2016 at 5:57 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
> > 
> > 
> > On Fri, 2016-02-26 at 17:26 +0100, Pavel Grunt wrote:
> > > On Fri, 2016-02-26 at 17:22 +0100, Fabiano Fidêncio wrote:
> > > > This is a temporary solution for using autobuild.sh, as commit
> > > > df403f5
> > > > introduced the -Wdeprecated-declarations and gtk-vnc provides
> > > > callbacks
> > > > using a deprecated structure (GValueArray), forcing virt-viewer to
> > > > keep
> > > > using deprecated methods (g_value_array_*).
> > > > 
> > > > Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> > > 
> > > Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx>
> > 
> > 
> > perhaps worth adding a comment to the code explaining why this is used
> > though?
> 
> (...) and gtk-vnc provides callbacks for getting authentication
> credentials which makes use of GValueArray, forcing virt-viewer to
> keep using deprecated methods like g_value_array_get_nth().
> 
> Does it work for you, Jonathon?


For me it can be as simple as:

/* gtk-vnc uses deprecated API, so disable warnings for this file */

But yours works too.


> 
> > 
> > 
> > > 
> > > > ---
> > > >  src/virt-viewer-session-vnc.c | 2 ++
> > > >  1 file changed, 2 insertions(+)
> > > > 
> > > > diff --git a/src/virt-viewer-session-vnc.c b/src/virt-viewer-session-
> > > > vnc.c
> > > > index 3321fc2..ccb08b2 100644
> > > > --- a/src/virt-viewer-session-vnc.c
> > > > +++ b/src/virt-viewer-session-vnc.c
> > > > @@ -24,6 +24,8 @@
> > > > 
> > > >  #include <config.h>
> > > > 
> > > > +#define GLIB_DISABLE_DEPRECATION_WARNINGS
> > > > +
> > > >  #include "virt-viewer-auth.h"
> > > >  #include "virt-viewer-session-vnc.h"
> > > >  #include "virt-viewer-display-vnc.h"
> > > 
> > > _______________________________________________
> > > 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




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux