[FFmpeg-devel] [PATCH v2] avformat/mxfenc: use a zero based continuity counter

[Marton Balint]

On Mon, 2 Mar 2020, Tomas Härdin wrote:

> lör 2020-02-29 klockan 21:06 +0100 skrev Marton Balint:
>> The standard does not seem to require the counter to be zero based, but some
>> checker tools (MyriadBits MXFInspect, Interra Baton) have validations against 0
>> start...
>> 
>> Fixes ticket #6781.
>> 
>> Signed-off-by: Marton Balint <cus at passwd.hu>
>> ---
>>  libavformat/mxfenc.c             | 2 +-
>>  tests/ref/fate/mxf-user-comments | 2 +-
>>  tests/ref/lavf/mxf               | 4 ++--
>>  3 files changed, 4 insertions(+), 4 deletions(-)
>> 
>> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
>> index 7ea47d7311..5e0dc0e889 100644
>> --- a/libavformat/mxfenc.c
>> +++ b/libavformat/mxfenc.c
>> @@ -2686,7 +2686,7 @@ static void mxf_write_system_item(AVFormatContext *s)
>>      avio_w8(pb, mxf->content_package_rate); // content package rate
>>      avio_w8(pb, 0x00); // content package type
>>      avio_wb16(pb, 0x00); // channel handle
>> -    avio_wb16(pb, (mxf->tc.start + frame) & 0xFFFF); // continuity count, supposed to overflow
>> +    avio_wb16(pb, frame & 0xFFFF); // continuity count, supposed to overflow
>
> OK I suppose

Thanks, applied.

Regards,
Marton