Re: [PATCH spice-server] sound: Store only playback channels in global list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 03, 2017 at 06:06:21AM -0400, Frediano Ziglio wrote:
> > 
> > On Wed, May 03, 2017 at 10:47:48AM +0100, Frediano Ziglio wrote:
> > > The list is used only to iterate playback channels
> > 
> > This patch seems only partial?
> > 
> > > 
> > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> > > ---
> > >  server/sound.c | 19 +++++++------------
> > >  1 file changed, 7 insertions(+), 12 deletions(-)
> > > 
> > > diff --git a/server/sound.c b/server/sound.c
> > > index be7e607..64cd6d4 100644
> > > --- a/server/sound.c
> > > +++ b/server/sound.c
> > > @@ -234,7 +234,7 @@ G_DEFINE_TYPE(RecordChannelClient,
> > > record_channel_client, TYPE_SND_CHANNEL_CLIEN
> > >  
> > >  
> > >  /* A list of all Spice{Playback,Record}State objects */
> 
> OT: this comment needs update too.. and perhaps not so OT.
> 
> > > -static GList *snd_channels;
> > > +static GList *snd_playback_channels;
> > >  
> > >  static void snd_send(SndChannelClient * client);
> > >  
> > > @@ -980,12 +980,10 @@ void snd_set_playback_latency(RedClient *client,
> > > uint32_t latency)
> > >  {
> > >      GList *l;
> > >  
> > > -    for (l = snd_channels; l != NULL; l = l->next) {
> > > +    for (l = snd_playback_channels; l != NULL; l = l->next) {
> > >          SndChannel *now = l->data;
> > 
> > The shortlog says l->data should be a SndChannelClient now?
> > 
> 
> ?? why you expect a client now? Neither in the subject not in commit
> log I said client, is and was a channel, not a channel client.

Oh whatever, I read "channel client" rather than "playback channel" in
the log.

/me goes to read the patch again.

Christopeh

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]