[FFmpeg-devel] [PATCH] mp4toannexb modification / improvement

Benoit Fouet benoit.fouet
Wed Jun 9 09:44:02 CEST 2010


Hi Luca,

On Fri, 04 Jun 2010 08:22:53 +0200 Luca Abeni wrote:
> On 06/03/2010 11:39 PM, Michael Niedermayer wrote:
> [...]
> >> Hence, I propose to modify / enhance this filter to return the
> >> converted extradata if no data is passed (that is, if buf_size
> >> is 0). The "regular" behaviour of the filter is not affected
> >> by this patch (the only difference is that if buf_size == 0 the
> >> filter returns the converted extradata instead of failing), and
> >> this patch allows to avoid a good amount of code duplication in
> >> sdp.c...
> >>
> >> Is the patch ok? If not, what is needed to make it acceptable?
> >
> > why does the bsf not convert the extradata?
> > i mean it leaves the extradata in "AVC" syntax
> 
> I do not know... My impression is that since the bsf "inlines"
> the SPS and PPS (inserting them in the bitstream) it does not care
> about converting the format of the extradata. But I am not sure;
> maybe Benoit can clarify this point?
> 

Sorry, I was offline last week.
Just for the record: I had written it that way just because I didn't
know bitstream filters were allowed to modify extradata.

Thanks for your patch.

Ben



More information about the ffmpeg-devel mailing list