[FFmpeg-devel] [PATCH] lavf: warn when a parser is needed and not found

Michael Niedermayer michaelni at gmx.at
Tue Aug 16 14:21:08 CEST 2011


On Tue, Aug 16, 2011 at 01:33:34PM +0200, Nicolas George wrote:
> lavf: warn when a parser is needed and not found.
> 
> At the very least, it should be useful for developers who disable a lot of
> thins to shorten the total build time and then wonder why it does not work.
> :-/
> 
> Regards,
> 
> -- 
>   Nicolas George

>  utils.c |    3 +++
>  1 file changed, 3 insertions(+)
> 42cbc946c5a72f5f75f010f8d23dd172045d4fcd  0001-lavf-warn-when-a-parser-is-needed-and-not-found.patch
> From 42063afd95c26aee1eecb62a6ab9aa684a1c96af Mon Sep 17 00:00:00 2001
> From: Nicolas George <nicolas.george at normalesup.org>
> Date: Tue, 16 Aug 2011 13:29:47 +0200
> Subject: [PATCH] lavf: warn when a parser is needed and not found.
> 
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavformat/utils.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index aa1220d..7c84c55 100644
> --- a/libavformat/utils.c
> +++ b/libavformat/utils.c
> @@ -1245,6 +1245,9 @@ static int read_frame_internal(AVFormatContext *s, AVPacket *pkt)
>              if (st->need_parsing && !st->parser && !(s->flags & AVFMT_FLAG_NOPARSE)) {
>                  st->parser = av_parser_init(st->codec->codec_id);
>                  if (!st->parser) {
> +                    av_log(s, AV_LOG_WARNING, "parser not found for codec "
> +                           "0x%x, packets or times may be invalid.\n",

with human readable codec name, LGTM

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110816/87e16db0/attachment.asc>


More information about the ffmpeg-devel mailing list