On Tue, May 24, 2011 at 09:31:30AM -0400, Ronald S. Bultje wrote: > Hi, > > On Tue, May 24, 2011 at 8:37 AM, Kostya <[email protected]> wrote: > > On Tue, May 24, 2011 at 02:14:44PM +0200, Anton Khirnov wrote: > >> From: XV <?@?> > >> > >> Signed-off-by: Anton Khirnov <[email protected]> > >> --- > >> libavcodec/intelh263dec.c | 8 ++++++-- > >> 1 files changed, 6 insertions(+), 2 deletions(-) > >> > >> diff --git a/libavcodec/intelh263dec.c b/libavcodec/intelh263dec.c > >> index c3d4d26..be6ce59 100644 > >> --- a/libavcodec/intelh263dec.c > >> +++ b/libavcodec/intelh263dec.c > >> @@ -46,7 +46,7 @@ int ff_intel_h263_decode_picture_header(MpegEncContext > >> *s) > >> skip_bits1(&s->gb); /* freeze picture release off */ > >> > >> format = get_bits(&s->gb, 3); > >> - if (format != 7) { > >> + if (format == 0 || format == 6) { > >> av_log(s->avctx, AV_LOG_ERROR, "Intel H263 free format not > >> supported\n"); > >> return -1; > >> } > >> @@ -64,7 +64,11 @@ int ff_intel_h263_decode_picture_header(MpegEncContext > >> *s) > >> s->obmc= get_bits1(&s->gb); > >> s->pb_frame = get_bits1(&s->gb); > >> > >> - if(format == 7){ > >> + if (format < 6) { > >> + s->width = h263_format[format][0]; > >> + s->height = h263_format[format][1]; > >> + } > >> + else { > > > > weird formatting > > > >> format = get_bits(&s->gb, 3); > >> if(format == 0 || format == 7){ > >> av_log(s->avctx, AV_LOG_ERROR, "Wrong Intel H263 format\n"); > >> -- > > > > otherwise looks ok > > Do we have fate samples for this?
nope, feel free to pick something from http://samples.multimedia.cx/A-codecs/IMC/ , the samples there have PB-frames which was a distinct feature of I263. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
