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

[Tomas Härdin]

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

/Tomas