Re: [FFmpeg-devel] [PATCH 1/6] Fix dshow device name/description

["Diederick C. Niehorster" <dcnieho@gmail.com>]

On Tue, Mar 22, 2022 at 3:10 PM Roger Pack <rogerdpack2@gmail.com> wrote:
>
> On Tue, Mar 22, 2022 at 7:40 AM <toots@rastageeks.org> wrote:
> >
> > From: Romain Beauxis <romain.beauxis@nextstep.com>
> >
> > diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c
> > index 6039578ff9..4ee3f6e194 100644
> > --- a/libavdevice/dshow.c
> > +++ b/libavdevice/dshow.c
> > @@ -552,8 +552,8 @@ dshow_cycle_devices(AVFormatContext *avctx, ICreateDevEnum *devenum,
> >                  if (!device)
> >                      goto fail;
> >
> > -                device->device_name = av_strdup(friendly_name);
> > -                device->device_description = av_strdup(unique_name);
> > +                device->device_name = av_strdup(unique_name);
> > +                device->device_description = av_strdup(friendly_name);
> >                  if (!device->device_name || !device->device_description)
> >                      goto fail;
> >
>
> LGTM.
> The device enumeration API was added only recently to dshow, guess a
> bug crept in.
> Thanks!

LGTM.

Indeed, this was code introduced by my patch, committed to the ffmpeg
repo only 91 days ago. Indeed, this should have been the other way
around given the documentation of struct AVDeviceInfo.

All the best,
Dee
_______________________________________________
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".