[FFmpeg-soc] vp8 de/packetizers

[Josh Allmann]

On 2 August 2010 06:20, Martin Storsjö <martin at martin.st> wrote:
> On Fri, 30 Jul 2010, Josh Allmann wrote:
>
>> On 30 July 2010 00:22, Martin Storsjö <martin at martin.st> wrote:
>> > On Thu, 29 Jul 2010, Josh Allmann wrote:
>> >
>> > [vp8 @ 0x10188fe00] Unknown profile 6
>> > [vp8 @ 0x10188fe00] Header size larger than data provided
>> > [rtsp @ 0x10202dc00] Received no start marker; dropping frame
>> >
>>
>> -noframedrop helped, but the last message is what I am concerned with;
>> it really should not happen. Those seem like phantom packets.
>
>
>> @@ -393,6 +418,8 @@ static int rtp_write_packet(AVFormatContext *s1, AVPacket *pkt)
>>      case CODEC_ID_H263P:
>>          ff_rtp_send_h263(s1, pkt->data, size);
>>          break;
>> +    case CODEC_ID_VP8:
>> +        rtp_send_vp8(s1, pkt->data, size);
>>      default:
>>          /* better than nothing : send the codec raw data */
>>          rtp_send_raw(s1, pkt->data, size);
>
> Take a look at this chunk and guess why you're getting phanton packets.
> :-)
>

1000l for not realizing this sooner.

> I'll take another look at the new version of the patches sometimes soon,
> but adding a break there seems to fix most of the issues :-)
>

Indeed, updated patches attached, along with the fixes requested by
Luca. Valgrind checks out cleanly.

Josh
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-RTP-packetization-of-VP8.patch
Type: text/x-patch
Size: 2702 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20100802/31b2176f/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Add-RTP-depacketization-of-VP8.patch
Type: text/x-patch
Size: 5626 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20100802/31b2176f/attachment-0001.bin>