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

[Diederick C. Niehorster]

On Tue, Apr 19, 2022 at 8:10 AM Roger Pack <rogerdpack2 at gmail.com> wrote:
>
> LGTM.

Not LGTM, see below

> On Wed, Apr 13, 2022 at 10:15 AM James Almer <jamrial at gmail.com> wrote:
> >
> > Remnant from f125c504d8fece6420bb97767f9e72414c26312a
> >
> > Signed-off-by: James Almer <jamrial at 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
> >              }

These should be used further down when checking audio formats. I'll
send a patch.

Cheers,
Dee