[FFmpeg-devel] [RFC] Subtitle Filtering Ramp-Up

[Michael Niedermayer]

Hi Nicolas

On Wed, Jun 04, 2025 at 09:13:44AM +0200, Nicolas George wrote:
> Zach Swena (HE12025-06-03):
[...]
>
> > This is why I have been trying to ask high level questions.  His old
> > subtitle filtering patchset would need a lot of rework to bring to the
> > current codebase so starting over isn't a bad idea.  I don't really care
> > who works on or makes the commits for the code as long as the resulting
> > code is clean, makes sense to other developers and accomplishes what
> > everyone needs.  There are structural changes needed for what I want and it
> > would be good to find a solution that allows for functionality for
> > additional processing options also.
> 
> His old filtering patchet was broken beyond repair, and nothing changed.
> 
> Just to give an idea of my position on the topic: during one of the
> first VDDs, possibly the first one, I co-hosted with Ubitux a
> multi-hours brainstorming session on the matter of subtitles in
> libavfilter. That is how much I want to keep subtitles out of
> libavfilter, and that is how little time I have spent on it anticipating
> problems and finding solution.
> 
> When I say that softworkz's approach is broken, I know what I am
> talking about.
> 
> It is broken in three ways.

Is it possible to create 3 testcases that one can just copy and paste to the
command line to see these 3 problems ?

If its possible, it would clearly show the problems and avoid a never
ending disagreement

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
than the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250604/2f6d1163/attachment.sig>