[Spice-devel] [PATCH spice-server 1a/3] red-channel: Use directly a GArray to pass capabilities

Frediano Ziglio fziglio at redhat.com
Tue Feb 28 17:14:25 UTC 2017


> 
> On Tue, Feb 28, 2017 at 09:43:33AM +0000, Frediano Ziglio wrote:
> > Capabilities where almost always passed using 2 arguments,
> > a number of elements and an array but then before using
> > these were converted to a GArray.
> > Converting to GArray much earlier allows to easily pass
> > the capabilities around.
> 
> Hey, nice improvement!
> 
> > 
> > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > ---
> > I don't know if is worth defining an abstract type
> > instead of using a simple GArray.
> > Maybe a "typedef GArray RedCapabilities" could help future
> > replacement of the GArray?
> 
> There's already a RedChannelCapabilities in red-channel.h, I'd reuse
> this one.
> 

Mumble... contains both common_caps and normal ones but can
be done, we always pass them together.
I was thinking about just joining caps+num.

> > On the other side this patch moves most of the array boxing
> > into a single function.
> 
> I'd create the GArray one level higher, in reds_channel_do_link() rather
> than red_channel_connect, let reds.c do the grunt work, and have
> RedChannel deal with higher level types.
> 
> Christophe
> 

If you want to reuse RedChannelCapabilities we could just box this
type and avoid the GArray change.
Or use GArray in a follow up (or not!).
Other work for reds.c? Is not enough the cacophonous stuff in here?

Frediano


More information about the Spice-devel mailing list