On Sun, Jan 15, 2023 at 03:24:37PM +0100, Jerome Martinez wrote:
> On 14/01/2023 21:04, Michael Niedermayer wrote:
> > On Sat, Jan 14, 2023 at 04:48:10PM +0100, Jerome Martinez wrote:
> > [...]
> > > +        stored_height = (stored_height+15)/16*16;
> > If this is supposed to match the actual macroblocks, then this would
> > have to consider field pictures and interlacing as it differs from
> > progressive
> 
> There is no change on that side, no addition, no change about it, this code
> is moved, and interlace is already managed in the current code:
> 
>     //Stored height
>     mxf_write_local_tag(s, 4, 0x3202);
>     avio_wb32(pb, stored_height>>sc->interlaced);
> [...]
>    //Display height
>     mxf_write_local_tag(s, 4, 0x3208);
>     avio_wb32(pb, display_height>>sc->interlaced);
> 
> The patch disables the rounding for some formats, it is not intended to
> change the behavior for MPEG-2 Video and AVC formats as it seems fine
> (FFmpeg has the same behavior as other muxers, at least for MPEG-2 Video,
> also for interlaced content).

ok

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad