[FFmpeg-devel] avformat/mxfenc: SMPTE RDD 48:2018 Amd 1:2022 (FFV1 in MXF) support
Tomas Härdin
git at haerdin.se
Wed Jan 18 15:40:39 EET 2023
Creating a new subthread because I just noticed something
> + //Stored height
> mxf_write_local_tag(s, 4, 0x3202);
> avio_wb32(pb, stored_height>>sc->interlaced);
>
Won't this be incorrect for files whose dimensions are multiples of 16
but not multiples of 32? Isn't each field stored separately with
dimensions a multiple of 16? So while for 1080p we'll have
StoredHeight = 1088
SampledHeight = 1080
and 1080i:
StoredHeight = 544
SampledHeight = 540
Where 544 is a multiple of 16, for say 720p we have
StoredHeight = 720
SampledHeight = 720
but for a hypothetical 720i we'd get
StoredHeight = 360
SampledHeight = 360
whereas the correct values should be
StoredHeight = 368
SampledHeight = 360
?
/Tomas
More information about the ffmpeg-devel
mailing list