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

[Paul B Mahol]

On 8/29/13, Robert Krueger <krueger at lesspain.de> 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.

Unrealated: log is using error level, but it should use warning level
(because error means that frame is not going to be
decoded/returned...),
also 'warning: ' then becomes superfluous.