[FFmpeg-devel] [PATCH 2/3] avformat/matroskaenc: write the FieldOrder element for non-interlaced video

James Almer jamrial at gmail.com
Wed Sep 28 00:14:36 EEST 2016


On 9/27/2016 5:56 PM, Hendrik Leppkes wrote:
> On Tue, Sep 27, 2016 at 8:03 PM, James Almer <jamrial at gmail.com> wrote:
>> It's listed as mandatory in https://matroska.org/technical/specs/index.html
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> The spec also mentions FieldOrder "MUST be ignored if FlagInterlaced is not
>> set to interlaced". Since it's a mandatory element, i interpreted that as a
>> demuxer guideline.
>>
> 
> Its a mandatory field with a default value - that kind of definition
> is a bit weird, but the general consensus is that it does not have to
> be written if it would write the default anyway.
> 
> - Hendrik

I find it weird it was defined this way, considering the precedent set by
BlockDuration which is not tagged as mandatory but its description
explicitly says "This Element is mandatory when DefaultDuration is set for
the track".

Alright then, patch dropped from the set.



More information about the ffmpeg-devel mailing list