[Ffmpeg-devel] Re: [Fwd: Re: DV Metadata Patch]

[Roman Shaposhnik]

On Wed, 2006-08-23 at 11:32 +0200, Baptiste Coudurier wrote:
> > +    /* Its hard to tell what SMPTE requires w.r.t. APT, but Quicktime needs it.
> > +     * We set it based on pix_fmt value but it really should be per DV profile */
> > +    int apt = (c->sys->pix_fmt == PIX_FMT_YUV422P ? 1 : 0);
> 
> Not that hard. Only YUV420P for pal (defined in IEC 61834) sets apt as
> 0. SMPTE 314M defines apt as 1.
 
  Fixed.

> Only YUV422P (DV50) has STYPE set to 1.

  Fixed.

Thanks,
Roman.