On date Wednesday 2024-04-03 21:50:29 +0200, Oneric wrote:
> On Mon, 19 Feb 2024 Oneric wrote:
> > avcodec/webvttdec: honour bidi marks
> > avcodec/{ass,webvttdec}: fix handling of backslashes
> > avcodec/{ass,webvttdec}: more portable curly brace escapes
> > avocdec/ass: simplify
On Mon, 19 Feb 2024 Oneric wrote:
> avcodec/webvttdec: honour bidi marks
> avcodec/{ass,webvttdec}: fix handling of backslashes
> avcodec/{ass,webvttdec}: more portable curly brace escapes
> avocdec/ass: simplify linebreaks
>
> libavcodec/ass.c | 47
Ping on this set authored by Oneric as well.
___
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".
Hi Oneric,
I was able to validate the whole set. Here are the tests I did ...
(1) test conversion from vtt->ass with bidi mark and visually check display
-- pass
(2) test slash escape fix with a string e.g. \\Nancy, verify broken before
and fixed by patch -- pass
(3) test curly brace escape fix
ping
I forgot to again add a description to v4 so:
These small patches fix several mis-conversions for subtitles
(and additionally one cde cleanup in avcodec/ass.c):
- WebVTT files no longer get their BiDi marks stripped
(which mangled BiDi text)
- backslashes are no longer duplicated for
Changes from v3:
- None. Just rebased ontop of master to allow
Patchwork to properly process the series now
that 99d33cc661fbd04e8657831 was merged.
Changes from v2:
- rebased ontop of the recently pushed eol normalisation.
As a result no more CRLFs in here and Patchwork should be