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

Tomas Härdin tjoppen at acc.umu.se
Thu Sep 14 15:30:26 EEST 2017


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?

/Tomas


More information about the ffmpeg-devel mailing list