On Wed, May 27, 2015 at 03:40:55PM +0200, Michael Niedermayer wrote: > This provides higher precission > > Signed-off-by: Michael Niedermayer <michae...@gmx.at> > --- > libavformat/mpsubdec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavformat/mpsubdec.c b/libavformat/mpsubdec.c > index c49f149..7c26d4f 100644 > --- a/libavformat/mpsubdec.c > +++ b/libavformat/mpsubdec.c > @@ -59,13 +59,13 @@ static int mpsub_read_header(AVFormatContext *s) > AVRational pts_info = (AVRational){ 100, 1 }; // ts based by default > int res = 0;
> int multiplier = 100; did you change that in a previous commit? > - float current_pts = 0; > + double current_pts = 0; > > av_bprint_init(&buf, 0, AV_BPRINT_SIZE_UNLIMITED); > > while (!avio_feof(s->pb)) { > char line[1024]; > - float start, duration; > + double start, duration; > int fps, len = ff_get_line(s->pb, line, sizeof(line)); > > if (!len) > @@ -77,7 +77,7 @@ static int mpsub_read_header(AVFormatContext *s) > /* frame based timing */ > pts_info = (AVRational){ fps, 1 }; > multiplier = 1; > - } else if (sscanf(line, "%f %f", &start, &duration) == 2) { > + } else if (sscanf(line, "%lf %lf", &start, &duration) == 2) { > AVPacket *sub; > const int64_t pos = avio_tell(s->pb); LGTM (no impact on fate i suppose?) -- Clément B.
pgpjailj946xS.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel