[FFmpeg-devel] [PATCH] RTP H.264 / AVC support
Sun Jun 27 00:10:51 CEST 2010
Luca Abeni wrote:
> On 06/16/2010 09:26 AM, Howard Chu wrote:
>>>>> After that, we can move to the rtpenc* part (yesterday I tried my
>>>>> ff_avc_find_startcode_avc() idea, and I think it can allow to simplify
>>>>> the code a little bit). In particular, I am still not sure about the
>>>>> need for the "NALP" data type.
>>>> Thanks for getting the sdp part fixed. What do we need to do now for
>>> I think you started discussing with Michael about exporting the NAL
>>> parsing code from h264.c, right?
>> Oh, actually I was waiting to see what idea you had first. You mentioned
>> above, an idea about ff_avc_find_startcode_avc(). You've already seen
>> what I came up with...
> It was just an idea... I attach a still-not-finished (works for me, but
> it's still unclean in some parts... In particular, I still need to reuse
> the avc_mp4_find_startcode() in other places) patch, which gives the idea.
> The good point about it is that it's simple.
>>> I was going to commit the rtpenc.* part of the patch (adding
>>> nal_length_size to struct RTPMuxContext), but then I noticed:
>>> s->nal_length_size = ((*(((char*)(st->codec->extradata))+4))&0x03)+1;
>>> and I am wondering why casting extradata to (char *)... I think this
>>> can be simplifed to
>>> s->nal_length_size = (st->codec->extradata& 0x03) + 1;
>>> what do you think?
>> Sounds good.
> Ok, good; I'll commit it later.
Ping... It's now much later; I'm getting lots of emails from Android users who
can't get ffserver working because this patch is missing. When are you
planning to commit it?
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/
More information about the ffmpeg-devel