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

2016-09-27 Thread James Almer
It's listed as mandatory in https://matroska.org/technical/specs/index.html Signed-off-by: James Almer --- 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. libavformat/matro

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

2016-09-27 Thread Hendrik Leppkes
On Tue, Sep 27, 2016 at 8:03 PM, James Almer wrote: > It's listed as mandatory in https://matroska.org/technical/specs/index.html > > Signed-off-by: James Almer > --- > The spec also mentions FieldOrder "MUST be ignored if FlagInterlaced is not > set to interlaced". Since it's a mandatory element

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

2016-09-27 Thread James Almer
On 9/27/2016 5:56 PM, Hendrik Leppkes wrote: > On Tue, Sep 27, 2016 at 8:03 PM, James Almer wrote: >> It's listed as mandatory in https://matroska.org/technical/specs/index.html >> >> Signed-off-by: James Almer >> --- >> The spec also mentions FieldOrder "MUST be ignored if FlagInterlaced is not

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

2016-09-27 Thread Hendrik Leppkes
On Tue, Sep 27, 2016 at 11:14 PM, James Almer wrote: > On 9/27/2016 5:56 PM, Hendrik Leppkes wrote: >> On Tue, Sep 27, 2016 at 8:03 PM, James Almer wrote: >>> It's listed as mandatory in https://matroska.org/technical/specs/index.html >>> >>> Signed-off-by: James Almer >>> --- >>> The spec also

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

2016-09-27 Thread James Almer
On 9/27/2016 6:36 PM, Hendrik Leppkes wrote: > On Tue, Sep 27, 2016 at 11:14 PM, James Almer wrote: >> On 9/27/2016 5:56 PM, Hendrik Leppkes wrote: >>> On Tue, Sep 27, 2016 at 8:03 PM, James Almer wrote: It's listed as mandatory in https://matroska.org/technical/specs/index.html Si