]> git.sesse.net Git - ffmpeg/commit
avcodec/movtextdec: Skip empty styles
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sat, 17 Oct 2020 15:48:54 +0000 (17:48 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Mon, 19 Oct 2020 03:02:00 +0000 (05:02 +0200)
commit40c16907bb69da6318917bbceac06f8325d361b6
treebb045d45dcbfcfc6e81db6b6f57b77a565e960c8
parent94ad68ee17420996c9b003f142717d82b52c0915
avcodec/movtextdec: Skip empty styles

They would either lead to unnecessary ASS tags being emitted (namely
tags that are reset immediately thereafter) or would lead to problems
when parsing: e.g. if a zero-length style immediately follows another
style, the current code will end the preceding style and set the
zero-length style as the next potentially active style, but it is only
tested for activation when the next character is parsed at which point
the current offset is already greater than both the starting as well
as the end offset of the empty style. It will therefore neither be
opened nor closed and all subsequent styles will be ignored.

Reviewed-by: Philip Langdale <philipl@overt.org>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavcodec/movtextdec.c