[FFmpeg-devel] [PATCH 2/2] avformat/genh: Remove unused array coef_splitted

Paul B Mahol onemda at gmail.com
Sun Oct 18 00:46:30 CEST 2015


On 10/17/15, Michael Niedermayer <michaelni at gmx.at> wrote:
> From: Michael Niedermayer <michael at niedermayer.cc>
>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/genh.c |    5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/libavformat/genh.c b/libavformat/genh.c
> index 3a4faf9..5d34491 100644
> --- a/libavformat/genh.c
> +++ b/libavformat/genh.c
> @@ -40,7 +40,6 @@ static int genh_read_header(AVFormatContext *s)
>  {
>      unsigned start_offset, header_size, codec, coef_type, coef[2];
>      GENHDemuxContext *c = s->priv_data;
> -    unsigned coef_splitted[2];
>      int align, ch;
>      AVStream *st;
>
> @@ -105,8 +104,8 @@ static int genh_read_header(AVFormatContext *s)
>      coef[1]          = avio_rl32(s->pb);
>      c->dsp_int_type  = avio_rl32(s->pb);
>      coef_type        = avio_rl32(s->pb);
> -    coef_splitted[0] = avio_rl32(s->pb);
> -    coef_splitted[1] = avio_rl32(s->pb);
> +    /*coef_splitted[0] =*/ avio_rl32(s->pb);
> +    /*coef_splitted[1] =*/ avio_rl32(s->pb);
>
>      if (st->codec->codec_id == AV_CODEC_ID_ADPCM_THP) {
>          if (st->codec->channels > 2) {
> --
> 1.7.9.5
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

I don't see point in doing this, when I find files  that make use of
those variables I will change it agaian.


More information about the ffmpeg-devel mailing list