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@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel