[FFmpeg-devel] [PATCH v3 0/7] Implementation of option one

Josh de Kock josh at itanimul.li
Mon Mar 19 21:35:34 EET 2018


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)



More information about the ffmpeg-devel mailing list