Re: [Spice-devel] [PATCH 07/14] Limit direct access to DisplayChannelClient

2016-05-12 Thread Jonathon Jongsma
On Thu, 2016-05-12 at 22:23 +0200, Pavel Grunt wrote: > On Thu, 2016-05-12 at 14:35 -0500, Jonathon Jongsma wrote: > > On Thu, 2016-05-05 at 14:28 +0200, Pavel Grunt wrote: > > > Hi Jonathon, > > > > > > there are some unrelated changes which should go in separate patch. > > > > > > > > > > [sn

Re: [Spice-devel] [PATCH 07/14] Limit direct access to DisplayChannelClient

2016-05-12 Thread Pavel Grunt
On Thu, 2016-05-12 at 14:35 -0500, Jonathon Jongsma wrote: > On Thu, 2016-05-05 at 14:28 +0200, Pavel Grunt wrote: > > Hi Jonathon, > > > > there are some unrelated changes which should go in separate patch. > > > > > > [snip] > > > > diff --git a/server/dcc.h b/server/dcc.h > > > index 509a6c

Re: [Spice-devel] [PATCH 07/14] Limit direct access to DisplayChannelClient

2016-05-12 Thread Jonathon Jongsma
On Thu, 2016-05-05 at 14:28 +0200, Pavel Grunt wrote: > Hi Jonathon, > > there are some unrelated changes which should go in separate patch. > > [snip] > > diff --git a/server/dcc.h b/server/dcc.h > > index 509a6c6..a56c4e2 100644 > > --- a/server/dcc.h > > +++ b/server/dcc.h > > @@ -18,11 +18

Re: [Spice-devel] [PATCH 07/14] Limit direct access to DisplayChannelClient

2016-05-05 Thread Pavel Grunt
Hi Jonathon, there are some unrelated changes which should go in separate patch. On Tue, 2016-05-03 at 15:00 -0500, Jonathon Jongsma wrote: > Add a few more methods and accessors so that other files don't need to > manipulate the struct members directly. Move the struct definition to a > private

[Spice-devel] [PATCH 07/14] Limit direct access to DisplayChannelClient

2016-05-03 Thread Jonathon Jongsma
Add a few more methods and accessors so that other files don't need to manipulate the struct members directly. Move the struct definition to a private header which only the dcc-* files will include. --- server/Makefile.am | 1 + server/dcc-encoders.c| 11 +++--- server/dcc-encoders.h