On Mon, Mar 28, 2016 at 06:09:03PM +0200, Clément Bœsch wrote: > On Mon, Mar 28, 2016 at 04:57:51PM +0200, Michael Niedermayer wrote: > > On Mon, Mar 28, 2016 at 02:10:51PM +0200, Clément Bœsch wrote: > > > lavc/utils already rescales avpkt->pts to sub->pts in AV_TIME_BASE_Q > > > before calling the decode callback. This prevents from rescaling again > > > into the decoder, and avoid the use of avctx->time_base which will > > > disappear in the incoming codecpar merge. > > > > > > This commit also replaces the use of "20 centisecond" (ass time base) > > > with "200 ms". > > > --- > > > libavcodec/ccaption_dec.c | 14 +++++++------- > > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > with > > ./ffmpeg -f lavfi -i "movie=Starship_Troopers.vob[out0+subcc]" -vn -map s > > out.srt > > > > this causes a moderate change in the timings > > is that intended ? > > > > Nope, that was a bug, thanks for noticing. It appears FATE isn't enough... > > New patch attached. > > [...] > > -- > Clément B.
> From 1c79debbb09c0ed8f1d771cc4224248351f4cea2 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= <u...@pkh.me> > Date: Mon, 28 Mar 2016 14:10:08 +0200 > Subject: [PATCH 1/2] lavc/ccaption_dec: remove usage of avctx->time_base > > lavc/utils already rescales avpkt->pts to sub->pts in AV_TIME_BASE_Q > before calling the decode callback. This prevents from rescaling again > into the decoder, and avoid the use of avctx->time_base which will > disappear in the incoming codecpar merge. > > This commit also replaces the use of "20 centisecond" (ass time base) > with "200 ms". > --- > libavcodec/ccaption_dec.c | 15 ++++++++------- > 1 file changed, 8 insertions(+), 7 deletions(-) > Applied. -- Clément B.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
