[Spice-devel] [client 0/3] Abstract video streaming from the network details

Francois Gouget fgouget at codeweavers.com
Tue Apr 4 15:45:22 UTC 2017


Currently the video decoders are directly passed network messages which 
ties them pretty closely to the network code. Should the protocol switch 
to a new type of messages the video decoding code will need to be 
updated. Even worse, it will have to deal with two types of network 
messages for backward compatiblity.

So I feel it's better to try and separate the two which is what I've 
done in the first two patches.

The third one does not really change the code much, it just separates 
the code that creates / destroys the display_stream objects from the 
code that handles the network messages leading to their creation / 
destruction.


Francois Gouget (3):
  streaming: Remove the display_stream dependency on SpiceMsgIn messages
  streaming: Remove the video decoder's dependency on SpiceMsgIn
    messages
  streaming: Separate the network code from the display_stream
    management

 src/channel-display-gst.c   | 128 +++++++++++++++-----------------
 src/channel-display-mjpeg.c |  74 +++++++++----------
 src/channel-display-priv.h  |  31 +++++---
 src/channel-display.c       | 176 ++++++++++++++++++++++----------------------
 4 files changed, 206 insertions(+), 203 deletions(-)

-- 
2.11.0


More information about the Spice-devel mailing list