[FFmpeg-devel] [PATCH v2 1/3] avcodec/libaribb24: change new lines to \n in ASS header

Jan Ekström jeebjp at gmail.com
Fri Aug 9 01:03:14 EEST 2024


On Fri, May 10, 2024 at 11:31 PM Kacper Michajłow <kasper93 at gmail.com> wrote:
>
> Fixes remaining \r\n is ASS header after 57c545090d.
>
> Signed-off-by: Kacper Michajłow <kasper93 at gmail.com>
> ---

With an initial look this set looks good. If I understand correctly,
the generic ASS encoder moved to outputting LF only in
7bf1b9b35769b37684dd2f18a54f01d852a540c8 and thus these modules which
still output manual headers with CRLF cause mismatching endlines
within a single document.

Set still applies. Will try to give this a proper look and verify
tomorrow, and then pull this in if it indeed does what it seems to do.

Jan


More information about the ffmpeg-devel mailing list