[FFmpeg-devel] [PATCH]Do not free non-allocated memory
Michael Niedermayer
michaelni
Sun Oct 18 22:55:53 CEST 2009
On Sun, Oct 18, 2009 at 09:54:03AM +0200, Reimar D?ffinger wrote:
> On Sat, Oct 17, 2009 at 10:11:23PM +0200, Michael Niedermayer wrote:
> > On Fri, Oct 16, 2009 at 05:40:47PM +0200, Reimar D?ffinger wrote:
> > > On Fri, Oct 16, 2009 at 06:14:11PM +0300, Kostya wrote:
> > > > On Fri, Oct 16, 2009 at 05:00:53PM +0200, Carl Eugen Hoyos wrote:
> > > > > Hi!
> > > > >
> > > > > As spotted by Alex Converse in a patch I sent recently. I believe his
> > > > > comment is also true for these files.
> > > > >
> > > > > Please comment, Carl Eugen
> > > >
> > > > Indeed, seems so.
> > > > Please apply.
> > >
> > > Maybe someone could be convinced to write a test program that just calls
> > > init() and close() or whatever it is called on all "things" (codecs,
> > > muxers,...) that would allow to easily spot such things with valgrind?
> >
> > and/or run the regression tests or fate under valgrind
>
> - will take really, really long, there is no way you can get people to run
> it before every commit, and running FATE like this would make it slow down
> enough that e.g. the AVR32 targets would be able to test far fewer revisions
> and with far more delay
i obviously did not mean to suggest that this is run for every revission in
fate or before every commit on the developers machine
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091018/752dca57/attachment.pgp>
More information about the ffmpeg-devel
mailing list