On date Monday 2011-05-30 21:15:16 +0200, Anton Khirnov encoded:
> Hi,
> since getting rid of AVFormatParameters is taking longer than I'd like,
> I've decided to dump this as an RFC so people can comment on API
> meanwhile.
> So here's the second take on reworking private options. It works by
> adding new av{codec,format,...}_open_*() functions which take an
> AVMetadata parameter. The user fills metadata with option names/values
> and passes it to the open call. The options are then automagically
> distributed to private contexts.
Is AVMetaData required at all (looks a bit overkill to me)? Also I'd
prefer to keep the current name scheme (av_metadata_*), "avmeta" is
both inconsistent and not very explicative ("meta" is too generic),
you can use the gnu ld hacks already employed for opt.c if you want to
avoid ABI breaks.
As alternative to metadata, what about using av_set_string_options(),
and pass a string with all the params?
> The advantages of this are:
> - the user doesn't need to care about global vs private options.
> - demuxer and decoder private options can be done easily.
> - it's conceivable to pass the options down from e.g. AVFormatContext
> to protocols.
>
> Note that the patchset doesn't yet pass fate, since we need replacements
> for all AVFormatParameters usage. Some polishing is also needed.
>
> This tree can be fetched from git://git.khirnov.net/git/libav, branch
> opts.
>
> Please comment.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
