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 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel