[FFmpeg-devel] [PATCH 1/4] avformat/mpeg: Remove set-but-unused variable

Paul B Mahol onemda at gmail.com
Tue Oct 8 10:23:24 EEST 2019


lgtm

On 10/8/19, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Forgotten in 7da57875.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavformat/mpeg.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
> index 6f132aae05..bd182e4429 100644
> --- a/libavformat/mpeg.c
> +++ b/libavformat/mpeg.c
> @@ -489,7 +489,6 @@ static int mpegps_read_packet(AVFormatContext *s,
>      MpegDemuxContext *m = s->priv_data;
>      AVStream *st;
>      int len, startcode, i, es_type, ret;
> -    int lpcm_header_len = -1; //Init to suppress warning
>      int pcm_dvd = 0;
>      int request_probe= 0;
>      enum AVCodecID codec_id = AV_CODEC_ID_NONE;
> @@ -507,8 +506,7 @@ redo:
>
>          if (!m->raw_ac3) {
>              /* audio: skip header */
> -            avio_r8(s->pb);
> -            lpcm_header_len = avio_rb16(s->pb);
> +            avio_skip(s->pb, 3);
>              len -= 3;
>              if (startcode >= 0xb0 && startcode <= 0xbf) {
>                  /* MLP/TrueHD audio has a 4-byte header */
> --
> 2.21.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".


More information about the ffmpeg-devel mailing list