Re: [FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-08 Thread Philip Langdale
On Mon, 6 Apr 2020 12:35:32 -0600 John Stebbins wrote: > The [Script Info] section was skipped if starts with UTF8 BOM > --- > libavcodec/ass_split.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c > index 67da7c6d84..c2c388d9f0 100644

[FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-06 Thread John Stebbins
The [Script Info] section was skipped if starts with UTF8 BOM --- libavcodec/ass_split.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index 67da7c6d84..c2c388d9f0 100644 --- a/libavcodec/ass_split.c +++ b/libavcodec/ass_split.c @@ -376,6 +37

Re: [FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-06 Thread Nicolas George
John Stebbins (12020-04-06): > Oh, whoops, I missed that ass_split gets called for a number of things. > This belongs at the beginning of ff_ass_split() I believe? I think so too. > In the sample I ran into this with, there's a BOM at the beginning of > the mkv private data for the track. I thin

Re: [FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-06 Thread John Stebbins
On Mon, 2020-04-06 at 20:08 +0200, Nicolas George wrote: > John Stebbins (12020-04-06): > > The [Script Info] section was skipped if starts with UTF8 BOM > > --- > > libavcodec/ass_split.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/libavcodec/ass_split.c b/libavcodec/ass_sp

Re: [FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-06 Thread Nicolas George
John Stebbins (12020-04-06): > The [Script Info] section was skipped if starts with UTF8 BOM > --- > libavcodec/ass_split.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c > index 67da7c6d84..94c32667af 100644 > --- a/libavcodec/ass_spli

[FFmpeg-devel] [PATCH 11/23] lavc/ass_split: fix parsing utf8 scripts

2020-04-06 Thread John Stebbins
The [Script Info] section was skipped if starts with UTF8 BOM --- libavcodec/ass_split.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index 67da7c6d84..94c32667af 100644 --- a/libavcodec/ass_split.c +++ b/libavcodec/ass_split.c @@ -354,6 +3