This set 'finishes' off the new API, it creates duct tape between the lavd and lavf libraries such that the groundwork for separating them is there, but lavd still requires lavf for the time-being. I have tagged this set with v3 (while they may not all be v3), to redirect review to this thread as it falls under the overarching discussion.
On 2018/03/19 18:08, Nicolas George wrote: > I do not understand what "in their current state" means? Before the > patches? After the patches? After half the patches but before the other > half? It's dependent on which route we take, options 2-4 wouldn't include this set as they would be replaced by other sets: 2) would change the way this patch has interactions between lavf and lavd. 3) would remove the interactions between lavf and lavd as they would just be the same library. 4) this set wouldnt exist. Josh de Kock (7): checkasm/Makefile: add EXTRALIBS-libavformat lavfi: add new iteration API cmdutils: use new iteration APIs fftools/ff*.c: use iteration API to find formats lavf,lavd: add AVClass for lavd & use iterate API lav*,tests: remove several register_all() calls lavf/img2dec: use new iteration API configure | 24 +- doc/APIchanges | 4 + doc/examples/filter_audio.c | 2 - doc/examples/filtering_audio.c | 2 - doc/examples/filtering_video.c | 2 - doc/examples/transcoding.c | 2 - doc/writing_filters.txt | 6 +- fftools/cmdutils.c | 249 ++++++------ fftools/ffmpeg.c | 6 - fftools/ffmpeg_opt.c | 23 +- fftools/ffplay.c | 28 +- fftools/ffprobe.c | 25 +- libavcodec/tests/utils.c | 1 - libavdevice/Makefile | 1 + libavdevice/avdevice.h | 1 + libavdevice/lavfi.c | 2 - libavdevice/options.c | 115 ++++++ libavfilter/allfilters.c | 818 ++++++++++++++++++++------------------- libavfilter/avfilter.c | 45 --- libavfilter/avfilter.h | 29 +- libavfilter/lavfutils.c | 2 - libavfilter/src_movie.c | 2 - libavfilter/tests/filtfmts.c | 2 - libavfilter/version.h | 3 + libavformat/img2dec.c | 3 +- libavformat/options.c | 46 ++- libavformat/tests/movenc.c | 2 - libavformat/tests/seek.c | 3 - tests/api/api-band-test.c | 2 - tests/api/api-codec-param-test.c | 2 - tests/api/api-flac-test.c | 2 - tests/api/api-h264-test.c | 2 - tests/api/api-seek-test.c | 2 - tests/checkasm/Makefile | 2 +- tools/aviocat.c | 1 - tools/enum_options.c | 2 - tools/graph2dot.c | 2 - tools/ismindex.c | 2 - tools/pktdumper.c | 3 - tools/probetest.c | 3 - tools/seek_print.c | 1 - tools/sidxindex.c | 2 - tools/uncoded_frame.c | 4 - 43 files changed, 795 insertions(+), 685 deletions(-) create mode 100644 libavdevice/options.c -- 2.14.3 (Apple Git-98) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel