]> git.sesse.net Git - ffmpeg/commit
avutil/timecode: fix av_timecode_get_smpte_from_framenum with 50/60 fps
authorMarton Balint <cus@passwd.hu>
Mon, 20 Jul 2020 19:38:21 +0000 (21:38 +0200)
committerMarton Balint <cus@passwd.hu>
Sun, 13 Sep 2020 15:51:57 +0000 (17:51 +0200)
commit00117e28c1133bf702a9adb8fa8cb3a356193bad
tree774f7c10f586e98167e011f44e9fb42dfaf6c6fd
parentddc9cce3cd20fdf8d523e4bab3cab55de6b862b0
avutil/timecode: fix av_timecode_get_smpte_from_framenum with 50/60 fps

SMPTE 12M timecode can only count frames up to 39, because the tens-of-frames
value is stored in 2 bit. In order to resolve this 50/60 fps SMPTE timecode is
using the field bit (which is the same bit as the phase correction bit) to
signal the least significant bit of a 50/60 fps timecode. See SMPTE ST
12-1:2014 section 12.1.

Therefore we slightly change the format of the return value of
av_timecode_get_smpte_from_framenum and AV_FRAME_DATA_S12M_TIMECODE and start
using the previously unused Phase Correction bit as Field bit. (As the SMPTE
standard suggests)

We add 50/60 fps support to av_timecode_get_smpte_from_framenum by calling the
recently added av_timecode_get_smpte function in it which already handles this
properly.

This change affects the decklink indev and the DV and MXF muxers. MXF has no
fate test for 50/60fps content, DV does, therefore the changes.

MediaInfo (a recent version) confirms that half-frame timecode must be inserted
to DV. MXFInspect confirms valid timecode insertion to the System Item of MXF
files. For MXF, also see EBU R122.

Note that for DV the field flag is not used because in the HDV specs (SMPTE
370M) it is still defined as biphase mark polarity correction flag. So it
should not matter that the DV muxer overrides the field bit.

Signed-off-by: Marton Balint <cus@passwd.hu>
libavutil/timecode.c
libavutil/timecode.h
tests/ref/vsynth/vsynth1-dv-hd
tests/ref/vsynth/vsynth2-dv-hd
tests/ref/vsynth/vsynth3-dv-hd
tests/ref/vsynth/vsynth_lena-dv-hd