Quoting Mark Thompson (2017-08-12 23:16:13) > Since there is no information about the source format, "unspecified" > is the correct value to write here. > --- > libavcodec/mpeg12enc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c > index 103f3aaa7..406950901 100644 > --- a/libavcodec/mpeg12enc.c > +++ b/libavcodec/mpeg12enc.c > @@ -297,7 +297,7 @@ static void mpeg1_encode_sequence_header(MpegEncContext > *s) > > put_header(s, EXT_START_CODE); > put_bits(&s->pb, 4, 2); // sequence > display extension > - put_bits(&s->pb, 3, 0); // video_format: > 0 is components > + put_bits(&s->pb, 3, 5); // video_format: > 5 is unspecified > put_bits(&s->pb, 1, 1); // > colour_description > put_bits(&s->pb, 8, s->avctx->color_primaries); // > colour_primaries > put_bits(&s->pb, 8, s->avctx->color_trc); // > transfer_characteristics > -- > 2.11.0
Sounds reasonable. I hope it doesn't break any crappy decoders. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
