[Spice-devel] [PATCH v6 04/26] server: Hide the MJPEG encoder internals from red_worker.c

[Christophe Fergeau]

On Wed, Oct 21, 2015 at 03:27:02PM +0200, Francois Gouget wrote:
> On Wed, 21 Oct 2015, Christophe Fergeau wrote:
> 
> > ACK.
> > 
> > Since the changes up to now are useful cleanups regardless of the
> > addition of gstreamer, I suggest we land the patches up to this one
> > right now, hopefully this will save some rebasing pain (haven't looked
> > at rest of the patches yet, I'll do that now).
> 
> Thanks!
> 
> There's a couple others like this further down the patch series for 
> spice-gtk:
> 
>  * [PATCH v6 20/26] spice-gtk: Prefix the configure audio GStreamer variables with GSTAUDIO_
>    http://lists.freedesktop.org/archives/spice-devel/2015-October/022402.html
> 
>  * [PATCH v6 21/26] spice-gtk: Enable adding alternative video decoders
>    This one actually received some comments and I split it in two and 
>    sent the updated patches as attachments to my reply. The first part, 
>    p21a.diff, is pretty short and would qualify as being independent 
>    from the GStreamer rework.
>    http://lists.freedesktop.org/archives/spice-devel/2015-October/022472.html
>    http://lists.freedesktop.org/archives/spice-devel/attachments/20151016/d2e54112/attachment.diff
> 
>    But sadly I have not gotten feedback on the modified patches yet so I 
>    don't know if they are really fit to go.

I haven't looked at the spice-gtk part of it either, but I've been at a
loss at applying these p21a/p21b/p23 patches to my tree, they did not
apply on top of patch21/patch23, nor seemed to be appropriate
replacements for these.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151021/13b4aa2a/attachment.sig>