[FFmpeg-devel] [PATCH] rawdec: do not unnecessarily allocate full-frame buffer

[Michael Niedermayer]

On Wed, Sep 22, 2010 at 07:14:58AM +0100, Reimar D?ffinger wrote:
> On Tue, Sep 21, 2010 at 09:52:24PM +0200, Michael Niedermayer wrote:
> > On Tue, Sep 21, 2010 at 07:15:53PM +0100, Reimar D?ffinger wrote:
> > > Hello,
> > > attached patch makes rawdec.c only allocate a buffer if it will
> > > actually be used (2 and 4 bpp formats), adding a separate palette
> > > to the context for use in the general PAL8 case.
> > > make test passes, and I don't think it should break anything,
> > > though I'll probably suggest a few more changes in a few days.
> > 
> > >  rawdec.c |   20 +++++++++++---------
> > >  1 file changed, 11 insertions(+), 9 deletions(-)
> > > bc1b224c82cf7e9e0b6b24d392a8faffcc827104  rawdec.diff
> > 
> > this should be ok if it fixes the bug and has been tested
> 
> Sorry I was unclear: No, this does not fix the valgrind bug,
> it just makes the patch for that simpler/less confusing.

hmm, seems mans reply confused me into thinking the patch did more than
was obvious
its ok still if tested ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100922/f42403bc/attachment.pgp>