From 20f972701806be20a77f808db332d9489343bb78 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Wed, 16 Mar 2022 21:09:54 +0100 Subject: avcodec/codec_internal: Add FFCodec, hide internal part of AVCodec Up until now, codec.h contains both public and private parts of AVCodec. This exposes the internals of AVCodec to users and leads them into the temptation of actually using them and forces us to forward-declare structures and types that users can't use at all. This commit changes this by adding a new structure FFCodec to codec_internal.h that extends AVCodec, i.e. contains the public AVCodec as first member; the private fields of AVCodec are moved to this structure, leaving codec.h clean. Reviewed-by: Anton Khirnov Signed-off-by: Andreas Rheinhardt --- libavcodec/pcx.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'libavcodec/pcx.c') diff --git a/libavcodec/pcx.c b/libavcodec/pcx.c index b8d911be6b..9c36e76343 100644 --- a/libavcodec/pcx.c +++ b/libavcodec/pcx.c @@ -25,6 +25,7 @@ #include "libavutil/imgutils.h" #include "avcodec.h" #include "bytestream.h" +#include "codec_internal.h" #include "get_bits.h" #include "internal.h" @@ -249,11 +250,11 @@ end: return ret; } -const AVCodec ff_pcx_decoder = { - .name = "pcx", - .long_name = NULL_IF_CONFIG_SMALL("PC Paintbrush PCX image"), - .type = AVMEDIA_TYPE_VIDEO, - .id = AV_CODEC_ID_PCX, +const FFCodec ff_pcx_decoder = { + .p.name = "pcx", + .p.long_name = NULL_IF_CONFIG_SMALL("PC Paintbrush PCX image"), + .p.type = AVMEDIA_TYPE_VIDEO, + .p.id = AV_CODEC_ID_PCX, .decode = pcx_decode_frame, - .capabilities = AV_CODEC_CAP_DR1, + .p.capabilities = AV_CODEC_CAP_DR1, }; -- cgit v1.2.3