Am Fr., 20. März 2020 um 01:49 Uhr schrieb Michael Niedermayer <mich...@niedermayer.cc>: > > On Thu, Mar 19, 2020 at 10:18:33PM +0100, Carl Eugen Hoyos wrote: > > Am Do., 19. März 2020 um 19:32 Uhr schrieb Michael Niedermayer > > <mich...@niedermayer.cc>: > > > > > > On Wed, Mar 18, 2020 at 02:16:30AM +0100, Carl Eugen Hoyos wrote: > > > > Hi! > > > > > > > > Attached patch fixes ticket #8575 for me, Google describes such files. > > > > > > > > Please comment, Carl Eugen > > > > > > > subviewerdec.c | 10 +++++++--- > > > > 1 file changed, 7 insertions(+), 3 deletions(-) > > > > 1a6133ed3dbb66945d6bf16e212c0e77e3fd0579 > > > > 0001-lavf-subviewerdec-Support-higher-sub-second-precisio.patch > > > > From 39d0748782bb3e37fb2f92c679ffa58b239374c7 Mon Sep 17 00:00:00 2001 > > > > From: Carl Eugen Hoyos <ceffm...@gmail.com> > > > > Date: Wed, 18 Mar 2020 02:11:33 +0100 > > > > Subject: [PATCH] lavf/subviewerdec: Support higher sub-second precision. > > > > > > > > Fixes ticket #8575. > > > > --- > > > > libavformat/subviewerdec.c | 10 +++++++--- > > > > 1 file changed, 7 insertions(+), 3 deletions(-) > > > > > > > > diff --git a/libavformat/subviewerdec.c b/libavformat/subviewerdec.c > > > > index 06b827b70f..ed48b3388d 100644 > > > > --- a/libavformat/subviewerdec.c > > > > +++ b/libavformat/subviewerdec.c > > > > @@ -56,11 +56,15 @@ static int read_ts(const char *s, int64_t *start, > > > > int *duration) > > > > int64_t end; > > > > int hh1, mm1, ss1, ms1; > > > > int hh2, mm2, ss2, ms2; > > > > + int multiplier = 1; > > > > > > > > + if (sscanf(s, "%u:%u:%u.%2u,%u:%u:%u.%2u", > > > > + &hh1, &mm1, &ss1, &ms1, &hh2, &mm2, &ss2, &ms2) == 8) > > > > + multiplier = 10; > > > > if (sscanf(s, "%u:%u:%u.%u,%u:%u:%u.%u", > > > > &hh1, &mm1, &ss1, &ms1, &hh2, &mm2, &ss2, &ms2) == 8) { > > > > > > which lengths can the ms1 / ms2 part have ? > > > do all of the following work ? > > > x.1 > > > x.19 > > > x.199 > > > x.1999 > > > > > > or are some of these guranteed not to occur ? > > > > I have no idea but my suspicion is that whoever created the files > > in question misunderstood the original format. > > then i would suggest that the code either supports all variants of this > or checks if the input is one of the supported ones. > basically checking how long the ms1/ms2 fields are IIUC
New patch attached. Thank you, Carl Eugen
From 17574e5cb44dead6990051540ada103610ee1e5b Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos <ceffm...@gmail.com> Date: Fri, 20 Mar 2020 09:02:59 +0100 Subject: [PATCH] lavf/subviewerdec: Support higher sub-second precision. Fixes ticket #8575. --- libavformat/subviewerdec.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/libavformat/subviewerdec.c b/libavformat/subviewerdec.c index 06b827b70f..83378eab5f 100644 --- a/libavformat/subviewerdec.c +++ b/libavformat/subviewerdec.c @@ -56,11 +56,21 @@ static int read_ts(const char *s, int64_t *start, int *duration) int64_t end; int hh1, mm1, ss1, ms1; int hh2, mm2, ss2, ms2; + int multiplier = 1; + if (sscanf(s, "%u:%u:%u.%2u,%u:%u:%u.%2u", + &hh1, &mm1, &ss1, &ms1, &hh2, &mm2, &ss2, &ms2) == 8) { + multiplier = 10; + } else if (sscanf(s, "%u:%u:%u.%1u,%u:%u:%u.%1u", + &hh1, &mm1, &ss1, &ms1, &hh2, &mm2, &ss2, &ms2) == 8) { + multiplier = 100; + } if (sscanf(s, "%u:%u:%u.%u,%u:%u:%u.%u", &hh1, &mm1, &ss1, &ms1, &hh2, &mm2, &ss2, &ms2) == 8) { - end = (hh2*3600LL + mm2*60LL + ss2) * 100LL + ms2; - *start = (hh1*3600LL + mm1*60LL + ss1) * 100LL + ms1; + ms1 = FFMIN(ms1, 999); + ms2 = FFMIN(ms2, 999); + end = (hh2*3600LL + mm2*60LL + ss2) * 1000LL + ms2 * multiplier; + *start = (hh1*3600LL + mm1*60LL + ss1) * 1000LL + ms1 * multiplier; *duration = end - *start; return 0; } @@ -84,7 +94,7 @@ static int subviewer_read_header(AVFormatContext *s) return res; if (avio_rb24(s->pb) != 0xefbbbf) avio_seek(s->pb, -3, SEEK_CUR); - avpriv_set_pts_info(st, 64, 1, 100); + avpriv_set_pts_info(st, 64, 1, 1000); st->codecpar->codec_type = AVMEDIA_TYPE_SUBTITLE; st->codecpar->codec_id = AV_CODEC_ID_SUBVIEWER; -- 2.24.1
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".