On Thu, Dec 01, 2016 at 11:47:15AM -0500, Frediano Ziglio wrote:
> > 
> > On Thu, Dec 01, 2016 at 11:24:26AM +0000, Frediano Ziglio wrote:
> > > @@ -1506,73 +1506,73 @@ static void remove_worker(SndWorker *worker)
> > >  
> > >  void snd_attach_playback(RedsState *reds, SpicePlaybackInstance *sin)
> > >  {
> > > -    SndWorker *playback_worker;
> > > -    RedChannel *channel;
> > > +    SndChannel *playback;
> > > +    RedChannel *red_channel;
> > 
> > Here s/channel/red_channel was not really necessary as the other
> > variable is named "playback" and not channel, ditto in snd_attach_record
> > below.
> > 
> 
> Should I revert them?

I'm fine with both options, so as you prefer :)

Christophe

Attachment: signature.asc
Description: PGP signature

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

Reply via email to