[Spice-devel] [client v2 1/3] streaming: Modify display_update_stream_report() to take a SpiceChannel

Francois Gouget fgouget at codeweavers.com
Thu Aug 18 15:57:55 UTC 2016


This makes it consistent with the other channel-display functions.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---

This patch makes sense on its own and can be applied even if the 
other two in the series are not.


 src/channel-display.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/channel-display.c b/src/channel-display.c
index 709b3d2..54df9c0 100644
--- a/src/channel-display.c
+++ b/src/channel-display.c
@@ -1221,10 +1221,10 @@ void stream_display_frame(display_stream *st, SpiceMsgIn *frame_msg,
  * if the report window is bigger */
 #define STREAM_REPORT_DROP_SEQ_LEN_LIMIT 3
 
-static void display_update_stream_report(SpiceDisplayChannel *channel, uint32_t stream_id,
+static void display_update_stream_report(SpiceChannel *channel, uint32_t stream_id,
                                          uint32_t frame_time, int32_t latency)
 {
-    SpiceDisplayChannelPrivate *c = channel->priv;
+    SpiceDisplayChannelPrivate *c = SPICE_DISPLAY_CHANNEL(channel)->priv;
     display_stream *st;
     guint64 now;
 
@@ -1232,7 +1232,7 @@ static void display_update_stream_report(SpiceDisplayChannel *channel, uint32_t
     g_return_if_fail(c->streams != NULL);
     g_return_if_fail(c->nstreams > stream_id);
 
-    st = channel->priv->streams[stream_id];
+    st = c->streams[stream_id];
     g_return_if_fail(st != NULL);
 
     if (!st->report_is_active) {
@@ -1257,7 +1257,7 @@ static void display_update_stream_report(SpiceDisplayChannel *channel, uint32_t
         now - st->report_start_time >= st->report_timeout ||
         st->report_drops_seq_len >= STREAM_REPORT_DROP_SEQ_LEN_LIMIT) {
         SpiceMsgcDisplayStreamReport report;
-        SpiceSession *session = spice_channel_get_session(SPICE_CHANNEL(channel));
+        SpiceSession *session = spice_channel_get_session(channel);
         SpiceMsgOut *msg;
 
         report.stream_id = stream_id;
@@ -1273,7 +1273,7 @@ static void display_update_stream_report(SpiceDisplayChannel *channel, uint32_t
             report.audio_delay = UINT_MAX;
         }
 
-        msg = spice_msg_out_new(SPICE_CHANNEL(channel), SPICE_MSGC_DISPLAY_STREAM_REPORT);
+        msg = spice_msg_out_new(channel, SPICE_MSGC_DISPLAY_STREAM_REPORT);
         msg->marshallers->msgc_display_stream_report(msg->marshaller, &report);
         spice_msg_out_send(msg);
 
@@ -1403,7 +1403,7 @@ static void display_handle_stream_data(SpiceChannel *channel, SpiceMsgIn *in)
      */
     st->video_decoder->queue_frame(st->video_decoder, in,  latency);
     if (c->enable_adaptive_streaming) {
-        display_update_stream_report(SPICE_DISPLAY_CHANNEL(channel), op->id,
+        display_update_stream_report(channel, op->id,
                                      op->multi_media_time, latency);
         if (st->playback_sync_drops_seq_len >= STREAM_PLAYBACK_SYNC_DROP_SEQ_LEN_LIMIT) {
             spice_session_sync_playback_latency(spice_channel_get_session(channel));
-- 
2.8.1



More information about the Spice-devel mailing list