[FFmpeg-devel] [PATCH v4 3/7] cmdutils: use new iteration APIs

Nicolas George george at nsup.org
Thu Mar 22 13:32:29 EET 2018


Josh de Kock (2018-03-22):
> There is always the option to just merge lavf and lavd. The state of them
> being sort-of merged, but not really, isn't very good and adds a lot of
> complexity especially in inter-library dependencies (which are unneeded if
> lavf and lavd are either merged or actually separate).

You are driving your reasoning the wrong way: you start from the
limitations of your new API, and based on what it can do you intent huge
changes to the project that affect user interface. It should be the
opposite: first decide on the user interface and general design, and
then make sure the API allow it.

For user interface, I state:

1. FFmpeg should allow users to select devices the same way as
   (de)muxers, because it allows them to use devices in a basic way even
   when applications are not explicitly prepared for them, that makes
   extra features for free.

Hence, I deduce:

2. All lavf APIs should treat devices the same way as (de)muxers.

And I still think that the better option is to revert the new API and
design a new new one, learning from the small mistakes of this one.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180322/773c51c4/attachment.sig>


More information about the ffmpeg-devel mailing list