On 7/21/2018 4:54 PM, Michael Niedermayer wrote: > On Sat, Jul 21, 2018 at 02:28:21PM -0300, James Almer wrote: >> And add it to the list of configurable components in Makefile, so changes to >> the file will be tracked and the usual warning to suggest re-running >> configure >> will be shown. >> >> Signed-off-by: James Almer <jamr...@gmail.com> >> --- >> Makefile | 1 + >> configure | 2 +- >> libavcodec/Makefile | 1 + >> libavcodec/parser.c | 85 ---------------------------------- >> libavcodec/parsers.c | 106 +++++++++++++++++++++++++++++++++++++++++++ >> 5 files changed, 109 insertions(+), 86 deletions(-) >> create mode 100644 libavcodec/parsers.c > > parser.c, parsers.c parser_list.c > > decode.c (no decoder.c, no decoders, no decoder_list.c, no parse.c) > encode.c (matching decode but differing from parser) > > seems a bit inconsistent, naming wise.
decode.c and encode.c are not component related files. They are the same as parser.c is now, containing API independent of available components. It still could use some cleaning, in any case. We have the following right now: allcodecs.c (list of decoders and encoders, plus related iteration/register API) codec_list.c (generated file for enabled decoders and encoders) decode.c (non iteration/register related decoder API functions) encode.c (non iteration/register related encoder API functions) allfilters.c (list of filters, plus related iteration/register API) filter_list.c (generated file for enabled filters) allformats.c (list of demuxers and muxers, plus related iteration/register API) demuxer_list.c (generated file for enabled demuxers) muxer_list.c (generated file for enabled muxers) alldevices.c (list of indevs and outdevs, plus related iteration/register API) indev_list.c (generated file for enabled indevs) outdev_list.c (generated file for enabled outdevs) protocols.c (list of protocols, plus related iteration/register API) protocol_list.c (generated file for enabled protocols) bitstream_filters.c (list of bsfs, plus related iteration/register API) bsf_list.c (generated file for enabled bsfs) bsf.c (non iteration/register related bsf API functions) bitstream_filter.c (deprecated bsf API, to be removed in a major bump) parsers.c (list of parsers, plus related iteration/register API) parser_list.c (generated file for enabled parsers) parser.c (non iteration/register related parser API functions) We could add the "all" prefix to the last three component list files. We could rename parser.c into parse.c to be in line with decode/encode.c. We could split the generated list of decoders and encoders into their own files. We could look into moving the deprecated bsf functions into the main bsf file and choose a consistent naming for them all (bsf is shorter and probably best), etc. I can look into doing some cleaning in this regard if we come to a consensus as to what should be done. This patch was mainly to get the file listing available parsers to trigger the Makefile message in case of changes, while making sure it only happens when new parsers are added instead of every time a bug is fixed in av_parser_parse2() or any other such function. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
