On Mon, Dec 15, 2014 at 1:45 PM, Luca Barbato <[email protected]> wrote: > On 08/12/14 15:27, Vittorio Giovara wrote: >> >> From: Marton Balint <[email protected]> >> >> Reviewed-by: Michael Niedermayer <[email protected]> >> Signed-off-by: Marton Balint <[email protected]> >> --- >> libavcodec/mpeg12enc.c | 2 +- >> tests/ref/lavf/mxf_d10 | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c >> index 5fc5e36..8f9292f 100644 >> --- a/libavcodec/mpeg12enc.c >> +++ b/libavcodec/mpeg12enc.c >> @@ -324,7 +324,7 @@ static void >> mpeg1_encode_sequence_header(MpegEncContext *s) >> put_bits(&s->pb, 1, 1); >> put_bits(&s->pb, 6, (uint32_t)((time_code / fps) % 60)); >> put_bits(&s->pb, 6, (uint32_t)((time_code % fps))); >> - put_bits(&s->pb, 1, !!(s->flags & CODEC_FLAG_CLOSED_GOP)); >> + put_bits(&s->pb, 1, !!(s->flags & CODEC_FLAG_CLOSED_GOP) || >> s->intra_only); >> put_bits(&s->pb, 1, 0); // broken link >> } >> } >> diff --git a/tests/ref/lavf/mxf_d10 b/tests/ref/lavf/mxf_d10 >> index 5864e8e..8a62bb0 100644 >> --- a/tests/ref/lavf/mxf_d10 >> +++ b/tests/ref/lavf/mxf_d10 >> @@ -1,3 +1,3 @@ >> -8cf467a910c84dd05db24848becba42e *./tests/data/lavf/lavf.mxf_d10 >> +96f933913835a439dd97144303dc8929 *./tests/data/lavf/lavf.mxf_d10 >> 5330989 ./tests/data/lavf/lavf.mxf_d10 >> ./tests/data/lavf/lavf.mxf_d10 CRC=0x4474d480 >> > > why intra_only doesn't set the flag when it is set?
I am not extremely sure, probably not to interfere with the flag parameter. Do you think it would be a more appropriate to do that? -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
