Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-29 Thread Frediano Ziglio
> > On Fri, 2017-04-28 at 05:32 -0400, Frediano Ziglio wrote: > > > > > > On Wed, 2017-04-26 at 06:33 -0400, Frediano Ziglio wrote: > > > > > > > > > > The base RedChannel already keeps a list of channel clients, so > > > > > there's > > > > > no need for the SndChannel to also keep track of the

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-28 Thread Jonathon Jongsma
On Fri, 2017-04-28 at 05:32 -0400, Frediano Ziglio wrote: > > > > On Wed, 2017-04-26 at 06:33 -0400, Frediano Ziglio wrote: > > > > > > > > The base RedChannel already keeps a list of channel clients, so > > > > there's > > > > no need for the SndChannel to also keep track of the client > > > > i

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-28 Thread Frediano Ziglio
> > On Wed, 2017-04-26 at 06:33 -0400, Frediano Ziglio wrote: > > > > > > The base RedChannel already keeps a list of channel clients, so > > > there's > > > no need for the SndChannel to also keep track of the client itself. > > > > > > Since the SndChannel only supports a single client (wherea

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-26 Thread Jonathon Jongsma
On Wed, 2017-04-26 at 06:33 -0400, Frediano Ziglio wrote: > > > > The base RedChannel already keeps a list of channel clients, so > > there's > > no need for the SndChannel to also keep track of the client itself. > > > > Since the SndChannel only supports a single client (whereas other > > chann

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-26 Thread Frediano Ziglio
> > The base RedChannel already keeps a list of channel clients, so there's > no need for the SndChannel to also keep track of the client itself. > > Since the SndChannel only supports a single client (whereas other > channels may have some partial support for multiple clients), I've > provided a

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-25 Thread Pavel Grunt
On Mon, 2017-04-24 at 12:09 -0500, Jonathon Jongsma wrote: > The base RedChannel already keeps a list of channel clients, so > there's > no need for the SndChannel to also keep track of the client itself. > > Since the SndChannel only supports a single client (whereas other > channels may have som

Re: [Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-25 Thread Christophe Fergeau
On Mon, Apr 24, 2017 at 12:09:41PM -0500, Jonathon Jongsma wrote: > The base RedChannel already keeps a list of channel clients, so there's > no need for the SndChannel to also keep track of the client itself. > > Since the SndChannel only supports a single client (whereas other > channels may hav

[Spice-devel] [PATCH v2 6/6] sound: don't store client in SndChannel

2017-04-24 Thread Jonathon Jongsma
The base RedChannel already keeps a list of channel clients, so there's no need for the SndChannel to also keep track of the client itself. Since the SndChannel only supports a single client (whereas other channels may have some partial support for multiple clients), I've provided a convenience fu