Re: [FFmpeg-devel] [PATCH v4 1/2] avcodec/ccaption_dec: don't print multiple \an and \pos tags

2024-03-25 Thread Marth64
This should be withheld until after 7.0. In the same bug report it appears that 1 user is making use of the erroneous tags via an external script for a valid purpose. I don’t want to break people’s workflow even though it’s invalid ASS. After 7.0 I will revive this patchset and with additional

Re: [FFmpeg-devel] [PATCH v4 1/2] avcodec/ccaption_dec: don't print multiple \an and \pos tags

2024-03-25 Thread Marth64
This also fixes https://trac.ffmpeg.org/ticket/10927 (recently reported by a user) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

[FFmpeg-devel] [PATCH v4 1/2] avcodec/ccaption_dec: don't print multiple \an and \pos tags

2024-03-23 Thread Marth64
Closed Captions decoder prints multiple \pos ASS tags per cue, and preceding the \pos tag is a fixed \an7 that fixes rendering position from the top left corner and enforces justification. Printing multiple \an and \pos tags in this way is invalid behavior, because only the first \pos tag in a