Re: [Spice-devel] [PATCH 07/19] Free dispatchers at exit

2016-02-24 Thread Jonathon Jongsma
On Fri, 2016-02-19 at 06:54 -0500, Frediano Ziglio wrote: > > > > From: Jonathon Jongsma > > > > --- > > server/red-dispatcher.c | 7 +++ > > server/red-dispatcher.h | 1 + > > server/reds.c | 2 ++ > > 3 files changed, 10 insertions(+) > > > > diff --git a/server/red-dispatcher.

Re: [Spice-devel] [PATCH 06/19] Convert Dispatcher and MainDispatcher to GObjects

2016-02-24 Thread Jonathon Jongsma
On Fri, 2016-02-19 at 06:54 -0500, Frediano Ziglio wrote: > > > > From: Jonathon Jongsma > > > > > > -struct MainDispatcher { > > -Dispatcher base; > > -SpiceCoreInterfaceInternal *core; > > -RedsState *reds; > > +G_DEFINE_TYPE(MainDispatcher, main_dispatcher, TYPE_DISPATCHER) > >

[Spice-devel] [PATCH] server: Remove an unnecessary cast in spice_replay_next_cmd()

2016-02-24 Thread Francois Gouget
Signed-off-by: Francois Gouget --- This is a small clean up of one of my older patches but will not change anything to how the code works. So it can safely be ignored if so desired. server/red-replay-qxl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/red-replay-q

[Spice-devel] [PATCH 2/2] server: Store the opaque pointer in VideoEncoderRateControlCbs

2016-02-24 Thread Francois Gouget
It has the same lifetime as the corresponding structure so this simplifies keeping track of it. Signed-off-by: Francois Gouget --- server/mjpeg-encoder.c | 13 + server/mjpeg-encoder.h | 4 +++- server/stream.c| 5 +++-- 3 files changed, 11 insertions(+), 11 deletions(-) d

[Spice-devel] [PATCH 1/2] server: Remove the width and height parameters of encode_frame()

2016-02-24 Thread Francois Gouget
They always match the size of the source bitmap area: * because for sized frames they are set from the source area, * and because otherwise the initial stream's size is that of regular frames by definition. Note also that we do not lose any flexibility since the source area parameter can be used

[Spice-devel] [PATCH] configure: Require spice-protocol-0.12.11 at least

2016-02-24 Thread Michal Privoznik
After 7a06efde we require SPICE_MSG_DISPLAY_GL_SCANOUT_UNIX symbol from spice-protocol (or value to be more precise). This has been however introduced in version 0.12.11 of the spice-protocol (commit 3fc2221e there). However, our configure check does not reflect this change. Fix it. Signed-off-by:

[Spice-devel] [PATCH] Update .gitignore files

2016-02-24 Thread Michal Privoznik
The first part updates the .gitignore file in the root directory because the file is obviously generated by automake but poison our working tree. The second part updates the .gitignore under server/tests/ directory because since 0db1137d we have a noinst test library libtest.a. However, like in pre