On Mon, 2 May 2016 19:31:17 +0200
Diego Biurrun <[email protected]> wrote:
> ---
> libavcodec/mmaldec.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/libavcodec/mmaldec.c b/libavcodec/mmaldec.c
> index 193df7e..4bbb32c 100644
> --- a/libavcodec/mmaldec.c
> +++ b/libavcodec/mmaldec.c
> @@ -815,11 +815,11 @@ static const AVOption options[]={
> .version = LIBAVUTIL_VERSION_INT, \
> };
>
> -#define FFMMAL_DEC(NAME, ID) \
> +#define FFMMAL_DEC(NAME, ID, LONG_NAME) \
> FFMMAL_DEC_CLASS(NAME) \
> AVCodec ff_##NAME##_mmal_decoder = { \
> .name = #NAME "_mmal", \
> - .long_name = NULL_IF_CONFIG_SMALL(#NAME " (mmal)"), \
> + .long_name = NULL_IF_CONFIG_SMALL(#LONG_NAME " (MMAL)"), \
> .type = AVMEDIA_TYPE_VIDEO, \
> .id = ID, \
> .priv_data_size = sizeof(MMALDecodeContext), \
> @@ -835,6 +835,6 @@ static const AVOption options[]={
> AV_PIX_FMT_NONE}, \
> };
>
> -FFMMAL_DEC(h264, AV_CODEC_ID_H264)
> -FFMMAL_DEC(mpeg2, AV_CODEC_ID_MPEG2VIDEO)
> -FFMMAL_DEC(vc1, AV_CODEC_ID_VC1)
> +FFMMAL_DEC(h264, AV_CODEC_ID_H264, H.264)
> +FFMMAL_DEC(mpeg2, AV_CODEC_ID_MPEG2VIDEO, MPEG-2)
> +FFMMAL_DEC(vc1, AV_CODEC_ID_VC1, VC-1)
I don't know. It's not a tragedy, but on the other hand I think adding
1 macro argument (and causing conflicts in the yet unapplied mpeg4
patch) is not worth the trouble just for a _slightly_ better name for a
rather obscure decoder wrapper.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
