[Spice-devel] [PATCH spice-gtk v3 3/7] spice-uri: Add missing include

Pavel Grunt pgrunt at redhat.com
Fri May 20 09:15:04 UTC 2016


On Fri, 2016-05-20 at 10:14 +0200, Victor Toso wrote:
> Hi,
> 
> On Thu, May 19, 2016 at 06:38:05PM +0200, Pavel Grunt wrote:
> > Related: rhbz#1335239
> > ---
> >  src/spice-uri.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/src/spice-uri.c b/src/spice-uri.c
> > index 8cf870d..3bdb502 100644
> > --- a/src/spice-uri.c
> > +++ b/src/spice-uri.c
> > @@ -22,6 +22,7 @@
> > 
> >  #include "spice-client.h"
> >  #include "spice-uri.h"
> > +#include "spice-uri-priv.h"
> 
> The -priv is internal and it always includes the non -priv one.
> I would change the include above spice-uri.h to spice-uri-priv.h

I would include both, but can remove the other one if you prefer.

Pavel

> 
> Acked-by: Victor Toso <victortoso at redhat.com>
> 
> >  
> >  /**
> >   * SECTION:spice-uri
> > -- 
> > 2.8.2
> > 
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel


More information about the Spice-devel mailing list