From 65e053271a98f7acf3ef6b412998cfcb44a8eef8 Mon Sep 17 00:00:00 2001 From: Martin Storsjö Date: Mon, 10 Dec 2012 01:10:42 +0200 Subject: rtpenc_vp8: Include the picture number in VP8 packets MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This makes it easier for receivers to decide what to do if data is lost. Refactor calculating the max payload size, to avoid hardcoding the header size in too many places, reducing the number of lines that have to be touched if the header is adjusted further. Signed-off-by: Martin Storsjö --- libavformat/rtpenc_vp8.c | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) (limited to 'libavformat/rtpenc_vp8.c') diff --git a/libavformat/rtpenc_vp8.c b/libavformat/rtpenc_vp8.c index 1730379874..d24c8bc40d 100644 --- a/libavformat/rtpenc_vp8.c +++ b/libavformat/rtpenc_vp8.c @@ -26,24 +26,30 @@ void ff_rtp_send_vp8(AVFormatContext *s1, const uint8_t *buf, int size) { RTPMuxContext *s = s1->priv_data; - int len, max_packet_size; + int len, max_packet_size, header_size; s->buf_ptr = s->buf; s->timestamp = s->cur_timestamp; - max_packet_size = s->max_payload_size - 1; // minus one for header byte - // no extended control bits, reference frame, start of partition, + // extended control bit set, reference frame, start of partition, // partition id 0 - *s->buf_ptr++ = 0x10; + *s->buf_ptr++ = 0x90; + *s->buf_ptr++ = 0x80; // Picture id present + *s->buf_ptr++ = s->frame_count++ & 0x7f; + // Calculate the number of remaining bytes + header_size = s->buf_ptr - s->buf; + max_packet_size = s->max_payload_size - header_size; + while (size > 0) { len = FFMIN(size, max_packet_size); memcpy(s->buf_ptr, buf, len); - ff_rtp_send_data(s1, s->buf, len+1, size == len); // marker bit is last packet in frame + // marker bit is last packet in frame + ff_rtp_send_data(s1, s->buf, len + header_size, size == len); size -= len; buf += len; - s->buf_ptr = s->buf; - *s->buf_ptr++ = 0; // payload descriptor + // Clear the partition start bit, keep the rest of the header untouched + s->buf[0] &= ~0x10; } } -- cgit v1.2.3