Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Add early support for colour elements

2016-03-04 Thread Michael Niedermayer
On Thu, Mar 03, 2016 at 11:23:53PM -0800, Neil Birkbeck wrote: > And yet another revision, where the syntax lists are actually terminated... applied thanks [..] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB it is not once nor twice but times without number that th

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Add early support for colour elements

2016-03-03 Thread Neil Birkbeck
And yet another revision, where the syntax lists are actually terminated... On Thu, Mar 3, 2016 at 12:09 PM, Neil Birkbeck wrote: > I've attached a slightly cleaner (more isolated) patch that moves the > parsing into a helper function. > > On Mon, Feb 29, 2016 at 5:15 PM, Neil Birkbeck > wrote:

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Add early support for colour elements

2016-03-03 Thread Neil Birkbeck
I've attached a slightly cleaner (more isolated) patch that moves the parsing into a helper function. On Mon, Feb 29, 2016 at 5:15 PM, Neil Birkbeck wrote: > Adding early support for a subset of the proposed colour elements > according to the latest version of spec: > https://mailarchive.ietf.org

[FFmpeg-devel] [PATCH] lavf/matroskadec: Add early support for colour elements

2016-02-29 Thread Neil Birkbeck
Adding early support for a subset of the proposed colour elements according to the latest version of spec: https://mailarchive.ietf.org/arch/search/?email_list=cellar&gbt=1&index=hIKLhMdgTMTEwUTeA4ct38h0tmE I've left out elements for pix_fmt related things as there still seems to be some discussio