[FFmpeg-devel] [PATCH 1/2] avcodec: add decode_error field to AVFrame

[Reimar Döffinger]

On 10 Jul 2012, at 23:57, Marton Balint <cus at passwd.hu> wrote:
> On Tue, 10 Jul 2012, Reimar Döffinger wrote:
> 
>> On Sun, Jul 08, 2012 at 06:12:08PM +0200, Marton Balint wrote:
>>> +    /**
>>> +     * decode error flag of the frame, set to a nonzero codec-specific error
>>> +     * code if the decoder produced a frame, but there were errors during
>>> +     * the decoding.
>> 
>> I don't like the codec specific error. I'd very much prefer this to be a
>> bit field, with at least some things pre-defined.
>> I don't mind a part being reserved for codec-specific stuff though.
> 
> I gave this a thought before sending the patch, but I could not think of too many generic decode errors, do you have some ideas?

The most important one, that actually has a real-world use are to distinguish between a bitstream error and a missing reference frame, since they tend to have significantly different artefacts.