Re: [PATCHv3 spice-gtk 01/14] compat: add strtok_r fallback

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

 




----- Original Message -----
> On Mon, Feb 17, 2014 at 10:35:40PM +0100, Marc-André Lureau wrote:
> > diff --git a/gtk/glib-compat.h b/gtk/glib-compat.h
> > index c30a735..62580e2 100644
> > --- a/gtk/glib-compat.h
> > +++ b/gtk/glib-compat.h
> > @@ -1,5 +1,8 @@
> >  /* -*- Mode: C; c-basic-offset: 4; indent-tabs-mode: nil -*- */
> >  /*
> > +   Copyright (C) 2012-2014 Red Hat, Inc.
> > +   Copyright © 1998-2009 VLC authors and VideoLAN
> > +
> >     This library is free software; you can redistribute it and/or
> >     modify it under the terms of the GNU Lesser General Public
> >     License as published by the Free Software Foundation; either
> > @@ -18,6 +21,7 @@
> >  
> >  #include <glib-object.h>
> >  #include <gio/gio.h>
> > +#include "config.h"
> 
> If you add config.h here, it has to go as the very first include.

well, if you don't need any of its define, that's not really required..

> 
> >  
> >  #if !GLIB_CHECK_VERSION(2,26,0)
> >  #define G_DEFINE_BOXED_TYPE(TypeName, type_name, copy_func, free_func)
> >  G_DEFINE_BOXED_TYPE_WITH_CODE (TypeName, type_name, copy_func, free_func,
> >  {})
> 
> > diff --git a/gtk/glib-compat.c b/gtk/glib-compat.c
> > index 9ffadbb..5714173 100644
> > --- a/gtk/glib-compat.c
> > +++ b/gtk/glib-compat.c
> > @@ -1,5 +1,8 @@
> >  /* -*- Mode: C; c-basic-offset: 4; indent-tabs-mode: nil -*- */
> >  /*
> > +   Copyright (C) 2012-2014 Red Hat, Inc.
> > +   Copyright © 1998-2009 VLC authors and VideoLAN
> > +
> >     This library is free software; you can redistribute it and/or
> >     modify it under the terms of the GNU Lesser General Public
> >     License as published by the Free Software Foundation; either
> > @@ -14,6 +17,7 @@
> >     License along with this library; if not, see
> >     <http://www.gnu.org/licenses/>.
> >  */
> >  
> > +#include <string.h>
> >  #include "glib-compat.h"
> 
> and glib-compat.h needs to go before string.h here.

why?

> 
> Christophe
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]