[FFmpeg-devel] [RFC] AVFilter

Michael Niedermayer michael at niedermayer.cc
Tue Oct 15 20:40:12 EEST 2024


On Tue, Oct 15, 2024 at 01:20:31PM -0300, James Almer wrote:
> On 10/15/2024 1:12 PM, Michael Niedermayer wrote:
> > Hi all
> > 
> > This is a quick RFC about peoples oppinions on AVFilter
> > 
> > The question: Should anyone be able to write a filter (which
> > other people can use) ?
> > Or should only we be able to add filters ?
> > 
> > (This relates to what parts of the API are publically accessable)
> > 
> > So whats the point of this RFC ?
> > 
> > If we want a public API that allows writing filters then work on the API
> > has to be compatible with that direction.
> > 
> > "compatible" for example would be, "Adding whats missing to the public API"
> > or "deprecating existing API and writing a better API that makes all needed
> > parts public"
> > 
> > "incompatible" for example is "making bits and pieces private that are needed
> > by filters
> > 
> > Some of the currently pending patches seem to fall into the "incompatible"

> No bits or fields are being made private in pending patches afaics.

[FFmpeg-devel] [PATCH 3/4] fftools/opt_common: stop accessing a private field

The only field this stops accessing is

-               filter->process_command                        ? 'C' : '.',

but id say this is supposed to be a public field, its needed to be
accessed by filters.

(its also defined in the main public header, yes below a line that was added later
 saying its private so its not really public but also not really private)

I think before these fields are moved into a private header, it is a good idea
to disscuss, if the community wants "creating filters" to be possible though the
API (as it was originally intended) or to only allow for us and noone else to add
filters
(thats why i send that RFC, iam sorry if i worded something in a confusing way, i just
 think we need to discuss the direction before starting to march)

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20241015/bb62d91c/attachment.sig>


More information about the ffmpeg-devel mailing list