Re: [FFmpeg-devel] [PATCH 1/2] avdevice/dshow: remove unused variables

From: Roger Pack <rogerdpack2@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 1/2] avdevice/dshow: remove unused variables
Date: Tue, 19 Apr 2022 00:10:28 -0600
Message-ID: <CAL1QdWdCJqtN5qj2VXnZg1ShMcmEjp-GrjxWmYzbZnOSrHy-UA@mail.gmail.com> (raw)
In-Reply-To: <20220413161515.6999-1-jamrial@gmail.com>

LGTM.

On Wed, Apr 13, 2022 at 10:15 AM James Almer <jamrial@gmail.com> wrote:
>
> Remnant from f125c504d8fece6420bb97767f9e72414c26312a
>
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>  libavdevice/dshow.c | 8 --------
>  1 file changed, 8 deletions(-)
>
> diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c
> index 1e69620880..ac8b64366f 100644
> --- a/libavdevice/dshow.c
> +++ b/libavdevice/dshow.c
> @@ -814,10 +814,6 @@ dshow_cycle_formats(AVFormatContext *avctx, enum dshowDeviceType devtype,
>                                                                      / ctx->requested_framerate.num : 0;
>      int requested_width                       = ctx->requested_width;
>      int requested_height                      = ctx->requested_height;
> -    // audio
> -    int requested_sample_rate                 = ctx->sample_rate;
> -    int requested_sample_size                 = ctx->sample_size;
> -    int requested_channels                    = ctx->channels;
>
>      if (IPin_QueryInterface(pin, &IID_IAMStreamConfig, (void **) &config) != S_OK)
>          return;
> @@ -854,10 +850,6 @@ dshow_cycle_formats(AVFormatContext *avctx, enum dshowDeviceType devtype,
>                      requested_framerate      = fmt_info->framerate;
>                      requested_width          = fmt_info->width;
>                      requested_height         = fmt_info->height;
> -                } else {
> -                    requested_sample_rate = fmt_info->sample_rate;
> -                    requested_sample_size = fmt_info->sample_size;
> -                    requested_channels    = fmt_info->channels;
>                  }
>                  av_free(fmt_info);  // free but don't set to NULL to enable below check
>              }
> --
> 2.35.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".