[FFmpeg-devel] [PATCH] remove warning when an mpeg stream starts with a non-I-frame

[Michael Niedermayer]

On Thu, Aug 29, 2013 at 10:27:09PM +0200, Robert Krüger wrote:
> On Thu, Aug 29, 2013 at 6:01 PM, Paul B Mahol <onemda at gmail.com> wrote:
> > On 8/29/13, Robert Krueger <krueger at lesspain.de> wrote:
> >> The warning as it is now produces a lot of noise for perfectly valid
> >> cases, i.e. for streams that start with b-frames (in decoding order)
> >> that only reference the first I-frame, which is the case, for example,
> >> for typical XDCAM streams.
> >
> > Isn't it better to check if frame is not I-frame or b-frame instead of
> > removing warning?
> 
> Updated patch attached. This is based on my understanding of the spec
> and assumption what the warning is for. I hope this is now the correct
> condition to specifically exclude the mentioned case. A file that has
> these properties is attached to trac ticket #479.

>  mpegvideo.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> ef0c3873a21e2f977d22f54639c69ea5d20a547a  improve_first_frame_mpeg_warning.patch
> From 3606cc587b63c28d04da1b7f2cf52cbacd323e48 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Robert=20Kr=C3=BCger?= <krueger at lesspain.de>
> Date: Thu, 29 Aug 2013 21:59:14 +0200
> Subject: [PATCH] avcodec/mpegvideo: exclude closed-gop leading b-frame case
>  from warning

fixed differently

Thanks for analyzing the problem

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20130830/267b9252/attachment.asc>