[FFmpeg-devel] [PATCH 2/2] avformat/mxfdec: use the common packet pts setter function for opatom files

Marton Balint cus at passwd.hu
Sat Sep 23 20:26:29 EEST 2017


On Thu, 14 Sep 2017, Tomas Härdin wrote:

> On 2017-09-13 21:31, Marton Balint wrote:
>>
>> On Fri, 8 Sep 2017, Michael Niedermayer wrote:
>>
>>> On Thu, Sep 07, 2017 at 05:11:40PM +0200, Marton Balint wrote:
>>>> Fixes ticket #6631.
>>>>
>>>> Signed-off-by: Marton Balint <cus at passwd.hu>
>>>> ---
>>>>  libavformat/mxfdec.c | 12 +++---------
>>>>  1 file changed, 3 insertions(+), 9 deletions(-)
>>>
>>> please add a fate this if you push this
>>
>> Unfortunately we only have a sample which is around 16 MB but it seems 
>> like an overkill to add such a big sample to the fate suite for such a 
>> minor feature. And I am also not very fond of truncating the file and 
>> adding truncated files to the fate suite... So unless somebody can 
>> provide an opAtom AVC intra file which is only a few frames long, I 
>> see no good way to fate test this. Do you have an idea how to proceed?
>
> Try having the user that reported the bug generate a smaller + lower res 
> sample?

These are P2 camera files, the reporter can't provide smaller files.

Anyway I pushed the patch series as is, fate test can be added later if 
somebody comes up with a similar but smaller intra-frame OpAtom file.

Regards,
Marton


More information about the ffmpeg-devel mailing list