Re: [Spice-devel] [PATCH 04/14] Replace RedChannel::clients with GList

2016-05-10 Thread Christophe Fergeau
Hey, This patch mostly looks good to me, a few minor comments: On Tue, May 03, 2016 at 03:00:20PM -0500, Jonathon Jongsma wrote: > Instead of using a Ring, use a GList to store the list of channel > clients. This allows us to iterate the clients without poking inside of > the client struct to get

[Spice-devel] [PATCH 04/14] Replace RedChannel::clients with GList

2016-05-03 Thread Jonathon Jongsma
Instead of using a Ring, use a GList to store the list of channel clients. This allows us to iterate the clients without poking inside of the client struct to get the channel_link. This is required in order to make the RedChannelClient struct private. --- server/display-channel.c | 64 ---