[FFmpeg-devel] [PATCH] avcodec/codec_internal: remove unnecessary avcodec.h include

[Tomas Härdin]

sön 2025-02-16 klockan 17:50 -0300 skrev James Almer:
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/codec_internal.h | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/codec_internal.h
> b/libavcodec/codec_internal.h
> index 5b2db74590..01033f4705 100644
> --- a/libavcodec/codec_internal.h
> +++ b/libavcodec/codec_internal.h
> @@ -22,7 +22,6 @@
>  #include <stdint.h>
>  
>  #include "libavutil/attributes.h"
> -#include "avcodec.h"
>  #include "codec.h"
>  #include "config.h"
>  
> @@ -268,7 +267,7 @@ typedef struct FFCodec {
>       * ff_default_get_supported_config() will be used.
> `out_num_configs` will
>       * always be set to a valid pointer.
>       */
> -    int (*get_supported_config)(const AVCodecContext *avctx,
> +    int (*get_supported_config)(const struct AVCodecContext *avctx,
>                                  const AVCodec *codec,
>                                  enum AVCodecConfig config,
>                                  unsigned flags,
> @@ -283,7 +282,7 @@ typedef struct FFCodec {
>   * For AVCODEC_CONFIG_COLOR_RANGE, the output will depend on the
> bitmask in
>   * FFCodec.color_ranges, with a value of 0 returning NULL.
>   */
> -int ff_default_get_supported_config(const AVCodecContext *avctx,
> +int ff_default_get_supported_config(const struct AVCodecContext
> *avctx,
>                                      const AVCodec *codec,
>                                      enum AVCodecConfig config,
>                                      unsigned flags,

$ ./configure && make clean && nice make -j3

> libavcodec/allcodecs.c: In function 'av_codec_init_static':
> libavcodec/allcodecs.c:940:41: error: 'AV_CODEC_CONFIG_PIX_FORMAT' undeclared (first use in this function)
>                                          AV_CODEC_CONFIG_PIX_FORMAT, 0,
>                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~
> libavcodec/allcodecs.c:940:41: note: each undeclared identifier is reported only once for each function it appears in
> libavcodec/allcodecs.c:940:41: error: type of formal parameter 3 is incomplete
> libavcodec/allcodecs.c:944:41: error: 'AV_CODEC_CONFIG_FRAME_RATE' undeclared (first use in this function)
>                                          AV_CODEC_CONFIG_FRAME_RATE, 0,
>                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~
> libavcodec/allcodecs.c:944:41: error: type of formal parameter 3 is incomplete
> libavcodec/allcodecs.c:950:41: error: 'AV_CODEC_CONFIG_SAMPLE_FORMAT' undeclared (first use in this function)
>                                          AV_CODEC_CONFIG_SAMPLE_FORMAT, 0,
>                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> libavcodec/allcodecs.c:950:41: error: type of formal parameter 3 is incomplete
> libavcodec/allcodecs.c:954:41: error: 'AV_CODEC_CONFIG_SAMPLE_RATE' undeclared (first use in this function)
>                                          AV_CODEC_CONFIG_SAMPLE_RATE, 0,
>                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> libavcodec/allcodecs.c:954:41: error: type of formal parameter 3 is incomplete
> libavcodec/allcodecs.c:958:41: error: 'AV_CODEC_CONFIG_CHANNEL_LAYOUT' undeclared (first use in this function)
>                                          AV_CODEC_CONFIG_CHANNEL_LAYOUT, 0,
>                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> libavcodec/allcodecs.c:958:41: error: type of formal parameter 3 is incomplete

/Tomas