On Sat, Jul 29, 2017 at 09:27:47PM +0200, Clément Bœsch wrote: > This commit switches off forced correct nesting of tags and only keeps > it for font tags. See long explanations in the code for the rationale. > > This results in various FATE changes which I'll explain here: > > - various swapping in font attributes, this is mostly noise due to the > old reverse stack way of printing them. The new one is more correct as > the last attribute takes over the previous ones. > > - unrecognized tags disappears > > - invalid tags that were previously displayed aren't anymore (instead, > we have a warning). This is better for the end user > > The main benefit of this commit is to be more tolerant to error, leading > to a better handling of badly nested tags or random wrong formatting for > the end user. > --- > libavcodec/htmlsubtitles.c | 199 > ++++++++++++++++++++++++++------------- > tests/ref/fate/sub-sami2 | 4 +- > tests/ref/fate/sub-srt | 14 +-- > tests/ref/fate/sub-srt-badsyntax | 4 +- > tests/ref/fate/sub-textenc | 14 +-- > tests/ref/fate/sub-webvttenc | 14 +-- > 6 files changed, 157 insertions(+), 92 deletions(-) >
patchset applied [...] -- Clément B.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
