[FFmpeg-devel] [RFC] GXF Patches - HD

[Reuben Martin]

On Saturday, July 20, 2013 11:43:59 PM Michael Niedermayer wrote:
> On Mon, Jun 17, 2013 at 03:24:21PM -0500, Reuben Martin wrote:
> > DV Aux Data & Test Updates
> > 
> >  gxfenc.c |   58
> >  ++++++++++++++++++++++++++++++++++++++++++----------------
> >  1 file changed, 42 insertions(+), 16 deletions(-)
> > 
> > cb589732dfe15d1c25e8ebfd9ad43ca752ab6d9f  03-GXF_DV-aux-data.patch
> > From d3fe02926248361d322768621ac927b63ac8b9e8 Mon Sep 17 00:00:00 2001
> > From: Reuben Martin <reuben.m at gmail.com>
> > Date: Wed, 12 Jun 2013 23:17:21 -0500
> > Subject: [PATCH 3/8] Added support for writing correct auxiliary data for
> > DV> 
> >  streams. Different aux data is written for DVCAM and DVPRO formats.
> 
> does this patch fix some testcase / ticket ?
> or said differently how can the fix be checked ?
> 

Sorry for the delay, I've been on vacation the past 2 weeks.

I'm basing this off of SMPTE 360-2009, clause 7.1.3.2.

It also states that anything that is not marked DVCAM (yuv420p) is to be 
assumed DVCPRO.

I've been checking this against Grass Valley's GXF parser and checker tool: 
http://www.gvgdevelopers.com/concrete/products/k2/tstream___gxf_file_parser_and_checker/

It's a little DOS based CLI app that can run under wine.

The text output from its parser has confirmed in my testing that the patches 
are working correctly.

-Reuben