[FFmpeg-devel] [PATCH v2] avcodec/libdav1d: fix setting AVFrame reordered_opaque

[James Almer]

On 10/22/2019 11:51 AM, Andrey Semashev wrote:
> On 2019-10-22 17:14, Andrey Semashev wrote:
>> On 2019-10-22 17:09, James Almer wrote:
>>> On 10/22/2019 11:01 AM, Andrey Semashev wrote:
>>>> On 2019-10-18 02:18, James Almer wrote:
>>>>> Actually reorder the values.
>>>>>
>>>>> Should effectively fix ticket #8300.
>>>>>
>>>>> Signed-off-by: James Almer <jamrial at gmail.com>
>>>>> ---
>>>>> Now unconditionally propagating the field, since checking its value is
>>>>> not correct usage of the field.
>>>>
>>>> James, do you still plan working on this patch?
>>>
>>> Yes, but i have no way to test it. Can you confirm the current
>>> implementation in the tree misbehaves, and that this approach
>>> corrects it?
>>
>> No, I don't have a test video file that would cause frame reordering.
>> At least I don't think any of my files cause it.
>>
>> I tried my v3 patch with the files I have, it worked as intended.
> 
> Actually, no, one of the files causes a delay for one frame, so I can
> test it. Your v2 and my v3 patches work (i.e. the decoded
> reordered_opaque lags behind input pts for one frame).

Ok, patch applied then. Thanks for testing.