[FFmpeg-devel] [PATCH] avformat/rawvideodec: Disable option accidentally added for v210(x)

[lance.lmwang at gmail.com]

On Sat, Jan 08, 2022 at 09:08:58PM +0100, Andreas Rheinhardt wrote:
> 41f213c3bf629d549400e935e7f123e6cfa959ab accidentally added
> an unused pixel_format option to the v210(x) demuxers.
> Remove it before it really becomes part of the API.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavformat/rawvideodec.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/rawvideodec.c b/libavformat/rawvideodec.c
> index 68547fc50f..3f92211af1 100644
> --- a/libavformat/rawvideodec.c
> +++ b/libavformat/rawvideodec.c
> @@ -126,8 +126,9 @@ static int rawvideo_read_packet(AVFormatContext *s, AVPacket *pkt)
>  #define OFFSET(x) offsetof(RawVideoDemuxerContext, x)
>  #define DEC AV_OPT_FLAG_DECODING_PARAM
>  static const AVOption rawvideo_options[] = {
> -    { "video_size", "set frame size", OFFSET(width), AV_OPT_TYPE_IMAGE_SIZE, {.str = NULL}, 0, 0, DEC },
> +    /* pixel_format is not used by the v210 demuxers. */
>      { "pixel_format", "set pixel format", OFFSET(pixel_format), AV_OPT_TYPE_STRING, {.str = "yuv420p"}, 0, 0, DEC },
> +    { "video_size", "set frame size", OFFSET(width), AV_OPT_TYPE_IMAGE_SIZE, {.str = NULL}, 0, 0, DEC },
>      { "framerate", "set frame rate", OFFSET(framerate), AV_OPT_TYPE_VIDEO_RATE, {.str = "25"}, 0, INT_MAX, DEC },
>      { NULL },
>  };
> @@ -175,7 +176,7 @@ const AVInputFormat ff_bitpacked_demuxer = {
>  static const AVClass v210_demuxer_class = {
>      .class_name = "v210(x) demuxer",
>      .item_name  = av_default_item_name,
> -    .option     = rawvideo_options,
> +    .option     = rawvideo_options + 1,
>      .version    = LIBAVUTIL_VERSION_INT,
>  };

LGTM.

>  
> -- 
> 2.32.0
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

-- 
Thanks,
Limin Wang