[FFmpeg-devel] [PATCH 2/2] avformat/mxfdec: use the common packet pts setter function for opatom files
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
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.
More information about the ffmpeg-devel