[FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

[Andreas Cadhalpun]

On 10.12.2016 17:55, Stefano Sabatini wrote:
> From ebc34da37648a07f25da94a1662c278c13ca7383 Mon Sep 17 00:00:00 2001
> From: Nicolas George <george at 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
> <stefasab at gmail.com>.
> 
> Signed-off-by: Nicolas George <george at 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