On 10/12/2016 6:22 PM, Dave Rice wrote: > >> On Oct 12, 2016, at 3:54 PM, James Almer <jamr...@gmail.com> wrote: >> >> The spec says >> >> 9: Interlaced with bottom field displayed first and top field stored first >> 14: Interlaced with top field displayed first and bottom field stored first >> >> And avcodec.h states >> >> AV_FIELD_TB, //< Top coded first, bottom displayed first >> AV_FIELD_BT, //< Bottom coded first, top displayed first >> >> Signed-off-by: James Almer <jamr...@gmail.com> >> --- >> libavformat/matroska.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavformat/matroska.h b/libavformat/matroska.h >> index 1ff76e6..15e401c 100644 >> --- a/libavformat/matroska.h >> +++ b/libavformat/matroska.h >> @@ -286,8 +286,8 @@ typedef enum { >> MATROSKA_VIDEO_FIELDORDER_UNDETERMINED = 2, >> MATROSKA_VIDEO_FIELDORDER_TT = 1, >> MATROSKA_VIDEO_FIELDORDER_BB = 6, >> - MATROSKA_VIDEO_FIELDORDER_BT = 9, >> - MATROSKA_VIDEO_FIELDORDER_TB = 14, >> + MATROSKA_VIDEO_FIELDORDER_TB = 9, >> + MATROSKA_VIDEO_FIELDORDER_BT = 14, >> } MatroskaVideoFieldOrder; >> >> typedef enum { >> -- >> 2.9.1 > > LGTM. Thanks for fixing this. It now aligns to the same list in QuickTime as > was intended in the spec. > > Dave Rice
Pushed, thanks. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel