Ronald S. Bultje <rsbultje <at> gmail.com> writes:

> >      switch (track->enc->colorspace) {
> >      case AVCOL_TRC_BT709:     avio_wb16(pb, 1); break;
> > +    case AVCOL_PRI_BT470BG:
> >      case AVCOL_PRI_SMPTE170M: avio_wb16(pb, 6); break;
> >      case AVCOL_PRI_SMPTE240M: avio_wb16(pb, 7); break;
> >      default:                  avio_wb16(pb, 2);
> 
> Am I the only one that thinks it's weird that we 
> read the SPC field as a combination/mixture of 
> TRC/PRI instead of SPC fields?

The patch was merged with the change to SPC, 
and I fixed the others.

Thank you, Carl Eugen

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to