[FFmpeg-devel] [PATCH] set subtitle codec time_base

Aurelien Jacobs aurel
Wed Nov 17 00:53:24 CET 2010


On Tue, Nov 16, 2010 at 05:49:22AM +0100, Michael Niedermayer wrote:
> On Mon, Nov 15, 2010 at 02:41:43PM +0100, Aurelien Jacobs wrote:
> > Hi,
> > 
> > Attached patch ensure that av_find_stream_info() will set
> > codec->time_base also for subtitle tracks. The MicroDVD subtitle
> > decoder will use this field.
> > MicroDVD stores subtitle start and end display time as frame number
> > so they need to be rescaled according to time_base to produce
> > timestamps in milliseconds in resulting ASS events.
> > 
> > Aurel
> >  utils.c |    3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 6aa06aaa76dc46b500f2afb778b48e0b6d1bf129  sub_codec_timebase.diff
> > commit 85508cbc1339ae20c640c05df1bb51030e090ba5
> > Author: Aurelien Jacobs <aurel at gnuage.org>
> > Date:   Fri Aug 20 21:55:41 2010 +0200
> > 
> >     also compute codec time_base for AVMEDIA_TYPE_SUBTITLE
> 
> lgtm

Applied.

Aurel



More information about the ffmpeg-devel mailing list