On 1/19/2019 3:50 PM, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
> ---
>  libavformat/hcom.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/hcom.c b/libavformat/hcom.c
> index 35515cc5b2..6d3d726fa5 100644
> --- a/libavformat/hcom.c
> +++ b/libavformat/hcom.c
> @@ -38,7 +38,7 @@ static int hcom_probe(AVProbeData *p)
>  static int hcom_read_header(AVFormatContext *s)
>  {
>      AVStream *st;
> -    unsigned data_size, rsrc_size, huffcount;
> +    av_unused unsigned data_size, rsrc_size, huffcount;
>      unsigned compresstype, divisor;
>      unsigned dict_entries;
>      int ret;

No, they should be removed and the relevant avio_rb32() replaced with
avio_skip() alongside a comment to document what those bytes represent.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to