Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-14 Thread Matthieu Bouron
On Tue, Dec 14, 2021 at 10:41:27AM +0100, Matthieu Bouron wrote: > On Tue, Nov 30, 2021 at 10:22:20AM +0100, Nicolas Gaullier wrote: > > Fix GC container ul. > > Fix GC element type both for the generic case and for OPAtom. > > > > Thanks to Philip de Nier > > for checking the values, especially

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-14 Thread Matthieu Bouron
On Tue, Nov 30, 2021 at 10:22:20AM +0100, Nicolas Gaullier wrote: > Fix GC container ul. > Fix GC element type both for the generic case and for OPAtom. > > Thanks to Philip de Nier > for checking the values, especially for OPAtom. > --- > libavformat/mxfenc.c | 8 ++-- > tests/ref/lavf

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-08 Thread Tomas Härdin
ons 2021-12-08 klockan 02:18 +0100 skrev Marton Balint: > > > On Wed, 8 Dec 2021, Tomas Härdin wrote: > > > fre 2021-12-03 klockan 09:38 + skrev Nicolas Gaullier: > > > > Please add a reference to the relevant SMPTE document in the > > > > comment, or perhaps at the list of references at the

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-08 Thread Paul B Mahol
This is unacceptable behavior for maintainer. On Wed, Dec 8, 2021 at 12:13 AM Tomas Härdin wrote: > fre 2021-12-03 klockan 09:38 + skrev Nicolas Gaullier: > > > Please add a reference to the relevant SMPTE document in the > > > comment, or perhaps at the list of references at the start of th

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-07 Thread Marton Balint
On Wed, 8 Dec 2021, Tomas Härdin wrote: fre 2021-12-03 klockan 09:38 + skrev Nicolas Gaullier: > Please add a reference to the relevant SMPTE document in the > comment, or perhaps at the list of references at the start of the > file > > /Tomas I have added the reference to ST2019-4 for

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-07 Thread Tomas Härdin
fre 2021-12-03 klockan 09:38 + skrev Nicolas Gaullier: > > Please add a reference to the relevant SMPTE document in the > > comment, or perhaps at the list of references at the start of the > > file > > > > /Tomas > > I have added the reference to ST2019-4 for "VC3 mapping", so that > should

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-03 Thread Nicolas Gaullier
>Please add a reference to the relevant SMPTE document in the comment, or >perhaps at the list of references at the start of the file > >/Tomas I have added the reference to ST2019-4 for "VC3 mapping", so that should be ok for generic standard files. It seems redundant for me, but if you want, I

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-12-01 Thread Tomas Härdin
tis 2021-11-30 klockan 10:22 +0100 skrev Nicolas Gaullier: Fix GC container ul. Fix GC element type both for the generic case and for OPAtom. Thanks to Philip de Nier for checking the values, especially for OPAtom. ---  libavformat/mxfenc.c  | 8 ++--  tests/ref/lavf/mxf_opatom | 2 +-  2 f

[FFmpeg-devel] [PATCH] avformat/mxfenc: fix DNxHD GC ULs

2021-11-30 Thread Nicolas Gaullier
Fix GC container ul. Fix GC element type both for the generic case and for OPAtom. Thanks to Philip de Nier for checking the values, especially for OPAtom. --- libavformat/mxfenc.c | 8 ++-- tests/ref/lavf/mxf_opatom | 2 +- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a