Re: [Spice-devel] [PATCH 02/10] Add DisplayChannelPrivate struct

2016-09-09 Thread Jonathon Jongsma
On Fri, 2016-09-09 at 09:37 +0200, Pavel Grunt wrote: > Hi, > > I guess the "1-element array trick" can be used to avoid leak of > "priv" in these patches as well. > > Pavel Indeed, I forgot to change these. Thanks. > > On Thu, 2016-09-08 at 11:52 -0500, Jonathon Jongsma wrote: > > > > Move

Re: [Spice-devel] [PATCH 02/10] Add DisplayChannelPrivate struct

2016-09-09 Thread Pavel Grunt
Hi, I guess the "1-element array trick" can be used to avoid leak of "priv" in these patches as well. Pavel On Thu, 2016-09-08 at 11:52 -0500, Jonathon Jongsma wrote: > Move all of the DisplayChannel data memembers into a private struct > to > encapsulate things better. This necessitated a few n

[Spice-devel] [PATCH 02/10] Add DisplayChannelPrivate struct

2016-09-08 Thread Jonathon Jongsma
Move all of the DisplayChannel data memembers into a private struct to encapsulate things better. This necessitated a few new 'public' methods and a small bit of refactoring to avoid poking into DisplayChannel internals from too many places. The DisplayChannel and the DisplayChannelClient are still