On Fri, 12 Aug 2016, Frediano Ziglio wrote:
[...]
> > +++ b/server/spice-server.h
> > @@ -22,8 +22,7 @@
> >  #error "Only spice.h can be included directly."
> >  #endif
> >  
> > -#include <glib.h>
> > -
> > +#include <spice/macros.h>
> >  #include "spice-core.h"
> 
> spice-core.h already include <spice/macros.h> so I would
> avoid to add the include here (as the other headers).


[...]
> Beside the include the patch is fine, if you agree I can change and
> merge.

Sure that works.


-- 
Francois Gouget <fgou...@codeweavers.com>
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to