On Thu, Apr 23, 2015 at 11:41 AM, Thilo Borgmann <thilo.borgm...@mail.de>
wrote:

> Am 23.04.15 um 11:07 schrieb Daniel Ly:
> > +    info->device_name = av_asprintf("[%d] %s: %s", index, description,
> model);
>
> So I assume that this is not parsed later.
> However, it changes current output format and should interfere with
> parse_device_string(). Consider choosing the very same format string which
> is
> there already.
>

Yes. The new patch with the retrofitted method produces exactly the same
output as before.


> > +    av_log(s->priv_data, AV_LOG_INFO, "%s\n", info->device_name);
>
> This always prints all devices, which is an unwanted behaviour.
> Currently you have to cause an error (e.g. wrong index) or explicitly call
> with
> -list_devices to get that.
>

Yes. avdevice_list_input_sources() is silent now.

Daniel
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to