[FFmpeg-devel] [PATCH] Function to parse Dirac sequence header

Diego Biurrun diego
Tue Dec 29 16:30:54 CET 2009


On Mon, Dec 28, 2009 at 02:10:07AM +0100, Michael Niedermayer wrote:
> On Wed, Nov 12, 2008 at 11:01:28PM -0500, David Conrad wrote:
> > On Nov 12, 2008, at 10:00 PM, Anuradha Suraparaju wrote:
> >>
> >> This is what I am doing in my local tree.
> >>
> >> in ffdirac_header
> >> -----------------
> >>
> >> st->need_parsing = AVSTREAM_PARSE_HEADERS;
> >> ( This ensures that the decoder is invoked during av_find_stream_info
> >> call)
> >>
> >> in ffdirac_gptopts
> >>
> >> if (st->codec->time_base.num && st->codec->time_base.den)
> >> 		st->time_base = st->codec->time_base;
> >>
> >> It appears to be an overkill to check the codec timebase and set the
> >> stream time base in every call to ffdirac_gptopts, but the check is
> >> required if we don't want to decode the Dirac sequence header in
> >> ffdirac_header.
> >
> > There's still the issue that the duration isn't set; this is something that 
> > imo should be done when possible, which it is for nearly all ogg files.
> >
> > At any rate I made some changes in the soc tree to this function. Updated 
> > version attached.
> 
> > commit b5dad233c8187289cfc674fd6a67ffdfe4734d73
> > Author: David Conrad <davedc at Kozue.local>
> > Date:   Wed Nov 12 22:42:47 2008 -0500
> > 
> >     Add a function to parse Dirac's sequence header
> 
> is this waiting for a review? applied? approved? rejected?
> or in need of a maintainer? if so someone should add himself to MAINTAINERS
> if no ffmpeg devel wants to maintain it i dont mind if someon from dirac
> takes over maintainership of our libdirac wraper related code

Shall I mail Anuradha?  Maybe she is willing to take over maintainership.

Diego



More information about the ffmpeg-devel mailing list