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

From: Pavel Koshevoy <pkoshevoy@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH] avformat/mov: Fix get_eia608_packet c608 caption reformatting
Date: Thu, 13 Feb 2025 14:05:39 -0700
Message-ID: <CAJgjuoyMJA6OQG4vsd6aL3FSiCaO3weQG4_dzS9L0udGth66Aw@mail.gmail.com> (raw)
In-Reply-To: <CAHGibzGNAdcL0xGrDbi0HfDO1U46O4aLKci59VXrrYoUL+YHCg@mail.gmail.com>

On Thu, Feb 13, 2025 at 1:12 PM Devin Heitmueller <
devin.heitmueller@ltnglobal.com> wrote:

> On Thu, Feb 13, 2025 at 3:00 PM Pavel Koshevoy <pkoshevoy@gmail.com>
> wrote:
> > yes, that patch is better in that it doesn't over-allocate.
> > I don't care much which fix gets merged, so long as the problem is fixed.
>
> Ok, if you could please try that patch and assuming it works for you
> reply to the list accordingly.  I tried it two years ago (which is how
> I found a bug in it), but it would be good to have someone else test
> it before it gets merged.

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.

Pavel.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".