[FFmpeg-devel] [PATCH] add support for "Forward Uncompressed" (issue1467)

[Reimar Döffinger]

On Tue, Oct 13, 2009 at 07:38:11PM +0200, Michael Niedermayer wrote:
> > > > +    if (avcodec_check_dimensions(avctx, avctx->width, avctx->height) < 0)
> > > > +        return -1;
> > > 
> > > dont we check that in common code already?
> > 
> > Yes, looks like it. A huge number of codecs seem to do this though.
> 
> Some do also need it, like the ones changing width/height

Yes, but among those that don't I think are
8bps.c
cscd.c
dxa.c
flashsvenc.c
lcldec.c
mmvideo.c
qdrw.c
qtrle.c
roqvideoenc.c
smacker.c
truemotion2.c
tscc.c
v210dec.c
v210x.c
vb.c
vmnc.c
xan.c
zmbv.c
zmbvenc.c

> > +static av_cold int decode_close(AVCodecContext *avctx)
> > +{
> > +    av_freep(&avctx->coded_frame);
> > +
> > +    return 0;
> > +}
> 
> missing release_buffer()

Hm... at least v210dec.c and v210x.c lack that, too...