Gaullier Nicolas <nicolas.gaullier <at> arkena.com> writes:

>      case 0x3212:
>          descriptor->field_dominance = avio_r8(pb);
> +        descriptor->field_dominance_present = 1;

Is it possible to instead initialize 
descriptor->field_dominance to MXF_TFF?

>                  case SeparateFields:
> +                    if (!descriptor->field_dominance_present)
> +                        descriptor->field_dominance = MXF_TFF;
>                      switch (descriptor->field_dominance) {
>                      case MXF_TFF:
>                          st->codec->field_order = AV_FIELD_TT;

Doesn't this allow to remove the default case below?

Carl Eugen

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

Reply via email to