On Thu, Apr 23, 2015 at 11:07:43AM +0200, Daniel Ly wrote: > This makes avdevice_list_input_sources() available for > device_name = "avfoundation". > > I didn't yet retrofit avf_read_header() to use the new function to > keep this patch small. I will post the follow-up patch in the same > thread. > > Signed-off-by: Daniel Ly <nal...@gmail.com> > --- > libavdevice/avfoundation.m | 98 > ++++++++++++++++++++++++++++++++++++++++++---- > 1 file changed, 90 insertions(+), 8 deletions(-) > > diff --git a/libavdevice/avfoundation.m b/libavdevice/avfoundation.m > index 763e675..19ce2a0 100644 > --- a/libavdevice/avfoundation.m > +++ b/libavdevice/avfoundation.m > @@ -34,6 +34,7 @@ > #include "libavformat/internal.h" > #include "libavutil/internal.h" > #include "libavutil/parseutils.h" > +#include "libavutil/avstring.h" > #include "libavutil/time.h" > #include "avdevice.h" > > @@ -1007,6 +1008,86 @@ static int avf_read_packet(AVFormatContext *s, > AVPacket *pkt) > return 0; > } > > +static int avf_add_device_info(AVDeviceInfoList *list, AVFormatContext *s, > + int index, const char *description, const char *model) > +{ > + AVDeviceInfo *info = av_mallocz(sizeof(AVDeviceInfo)); > + if (!info) return AVERROR(ENOMEM); > + > + info->device_name = av_asprintf("[%d] %s: %s", index, description, > model); > + info->device_description = strdup(description); > + if (!info->device_name || !info->device_description) { > + av_free(info); > + return AVERROR(ENOMEM); > + } > +
> + av_log(s->priv_data, AV_LOG_INFO, "%s\n", info->device_name); Why do you log this? Won't this cause a dup of the list in the output? [...] -- Clément B.
pgpKCQXV4szYr.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel