[FFmpeg-devel] [PATCH] avformat/mov: Fix get_eia608_packet c608 caption reformatting

[Devin Heitmueller]

On Thu, Feb 13, 2025 at 4:06 PM Pavel Koshevoy <pkoshevoy at gmail.com> wrote:
> I've submitted a 3rd version of the patch that doesn't do as much
> over-allocation, similar to
> https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2023-June/310456.html
> My v3 patch works correctly with the 2 sample files I can test with.

Your patch looks substantially more complicated than the one from two
years ago.  Does it address some edge case(s) that Sebastian's patch
didn't handle?

If your patch is better in some way then so be it.  But if it's
functionally equivalent I would suggest we merge Sebastian's patch as
it's simpler.

Devin

-- 
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com  e: devin.heitmueller at ltnglobal.com