[Spice-devel] [PATCH 00/15] Backported some patches from refactory branches (9th Nov)

[Jonathon Jongsma]

On Mon, 2015-11-09 at 13:15 +0000, Frediano Ziglio wrote:
> This patchset supersed last patchset.
> 
> Changes:
> - rebased on new master;
> - dropper "server/red_worker: red_draw_qxl_drawable: protect from
>   NULL dereference in case of buggy driver (or recording)" patch;
> - updated "worker: move red_destroy_surface_item()" checking
>   for dcc NULL before using it;
> - removed merged patches;
> - added some patches to the set.
> 
> I didn't split "worker: move stream to display channel" (waiting
> Jonathon).

Actually, I suggested that it probably wasn't worth splitting. But
there was one behavior change in then patch that concerned me. I would
appreciate a second pair of eyes on that section to make sure I was
reading it correctly.



> 
> Christophe Fergeau (1):
>   worker: simplify RedCompressBuf
> 
> Marc-André Lureau (14):
>   worker: move stream to display channel
>   worker: move red_destroy_surface_item()
>   worker: move image cache to display
>   worker: move current to display
>   worker: move drawable to display
>   worker: move surfaces
>   worker: make more functions static, kill dead function
>   worker: group add_memslot
>   worker: remove cursor channel asserts
>   worker s/surfaces_dest/surface_deps
>   worker: move compression parameters to dcc
>   worker: don't use weird RedCompressedBuf nbytes shifting
>   worker: move encoders to dcc-encoders
>   worker: rename {put,ref}_red_drawable
> 
>  server/Makefile.am         |    4 +
>  server/cursor-channel.c    |    6 +-
>  server/dcc-encoders.c      |  402 ++++++
>  server/dcc-encoders.h      |  145 ++
>  server/display-channel.c   |  174 ++-
>  server/display-channel.h   |  345 +++--
>  server/red_parse_qxl.c     |   10 +-
>  server/red_parse_qxl.h     |    2 +-
>  server/red_worker.c        | 3196 ++++++++++++++--------------------
> ----------
>  server/red_worker.h        |    3 +
>  server/spice_image_cache.c |   60 +
>  server/spice_image_cache.h |   19 +-
>  server/stream.c            |  136 ++
>  server/stream.h            |  148 ++
>  server/tree.h              |    6 +
>  server/utils.h             |    1 +
>  16 files changed, 2334 insertions(+), 2323 deletions(-)
>  create mode 100644 server/dcc-encoders.c
>  create mode 100644 server/dcc-encoders.h
>  create mode 100644 server/stream.c
>  create mode 100644 server/stream.h
>