On 10.12.2016 17:55, Stefano Sabatini wrote: > From ebc34da37648a07f25da94a1662c278c13ca7383 Mon Sep 17 00:00:00 2001 > From: Nicolas George <geo...@nsup.org> > Date: Sat, 11 Jan 2014 19:42:41 +0100 > Subject: [PATCH] lavf: add ffprobe demuxer > > With several modifications and documentation by Stefano Sabatini > <stefa...@gmail.com>. > > Signed-off-by: Nicolas George <geo...@nsup.org> > --- > doc/demuxers.texi | 19 ++ > doc/ffprobe-format.texi | 121 +++++++++++++ > doc/formats.texi | 1 + > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/ffprobedec.c | 452 > +++++++++++++++++++++++++++++++++++++++++++++++ > 6 files changed, 595 insertions(+) > create mode 100644 doc/ffprobe-format.texi > create mode 100644 libavformat/ffprobedec.c > [...] > diff --git a/libavformat/ffprobedec.c b/libavformat/ffprobedec.c > new file mode 100644 > index 0000000..f5d5ed7 > --- /dev/null > +++ b/libavformat/ffprobedec.c [...] > + if (av_strstart(buf, "codec_name=", &val)) { > + const AVCodecDescriptor *desc = > avcodec_descriptor_get_by_name(val); > + if (desc) { > + st->codecpar->codec_id = desc->id; > + st->codecpar->codec_type = desc->type; > + } > + if (!desc) { > + av_log(avf, AV_LOG_WARNING, "Cannot recognize codec name > '%s'", val);
This log message is missing a newline at the end. Other than that this only needs adding the format to doc/general.texi, a minor version bump and a changelog entry. Best regards, Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel