Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-12-06 Thread Frediano Ziglio
> > On 12/04/2017 06:07 PM, Frediano Ziglio wrote: > >> > >> On 11/29/2017 10:16 PM, Jonathon Jongsma wrote: > >>> This allows us to unref the stream directly rather than needing to pass > >>> the associated DisplayChannel to stream_unref(). The same is also true > >>> for stream_agent_unref, sinc

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-12-05 Thread Uri Lublin
On 12/04/2017 06:07 PM, Frediano Ziglio wrote: On 11/29/2017 10:16 PM, Jonathon Jongsma wrote: This allows us to unref the stream directly rather than needing to pass the associated DisplayChannel to stream_unref(). The same is also true for stream_agent_unref, since the only reason that stream

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-12-04 Thread Frediano Ziglio
> > On 11/29/2017 10:16 PM, Jonathon Jongsma wrote: > > This allows us to unref the stream directly rather than needing to pass > > the associated DisplayChannel to stream_unref(). The same is also true > > for stream_agent_unref, since the only reason that stream_agent_unref() > > required a Disp

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-12-03 Thread Uri Lublin
On 11/29/2017 10:16 PM, Jonathon Jongsma wrote: This allows us to unref the stream directly rather than needing to pass the associated DisplayChannel to stream_unref(). The same is also true for stream_agent_unref, since the only reason that stream_agent_unref() required a DisplayChannel paramete

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-12-01 Thread Frediano Ziglio
> > On Thu, 2017-11-30 at 13:40 -0500, Frediano Ziglio wrote: > > > > > > This allows us to unref the stream directly rather than needing to > > > pass > > > the associated DisplayChannel to stream_unref(). The same is also > > > true > > > for stream_agent_unref, since the only reason that > > >

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-11-30 Thread Jonathon Jongsma
On Thu, 2017-11-30 at 13:40 -0500, Frediano Ziglio wrote: > > > > This allows us to unref the stream directly rather than needing to > > pass > > the associated DisplayChannel to stream_unref(). The same is also > > true > > for stream_agent_unref, since the only reason that > > stream_agent_unref

Re: [Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-11-30 Thread Frediano Ziglio
> > This allows us to unref the stream directly rather than needing to pass > the associated DisplayChannel to stream_unref(). The same is also true > for stream_agent_unref, since the only reason that stream_agent_unref() > required a DisplayChannel parameter was to pass it to stream_unref(). > T

[Spice-devel] [PATCH spice-server 2/7] VideoStream: store channel in stream

2017-11-29 Thread Jonathon Jongsma
This allows us to unref the stream directly rather than needing to pass the associated DisplayChannel to stream_unref(). The same is also true for stream_agent_unref, since the only reason that stream_agent_unref() required a DisplayChannel parameter was to pass it to stream_unref(). This also resu