Re: [PATCH 1/2] Introduce leaky().

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 24, 2008 at 09:28:30PM +0000, Jakub Narebski wrote:
> Pierre Habouzit <madcoder@xxxxxxxxxx> writes:
> 
> > diff --git a/Makefile b/Makefile
> 
> > +#
> > +# Define COLLECT_LEAKS_AT_EXIT if you want memory marked as leaky() at exit.
> 
> I think s/at exit/to be freed &/;

  err obviously.

> > diff --git a/cache.h b/cache.h
> 
> Hmmm... cache?

  well cache.h has the prototypes for alloc.c that feels like the proper
place, but I don't care much :)

> >  /* alloc.c */
> > +#ifdef COLLECT_LEAKS_AT_EXIT
> > +extern void *leaky(void *);
> > +#else
> > +# define leaky(x) x
> > +#endif
> 
> Not 
> 
> +# define leaky(x) (x)
> 
> to be careful?

ack.

-- 
·O·  Pierre Habouzit
··O                                                madcoder@xxxxxxxxxx
OOO                                                http://www.madism.org

Attachment: pgpUam2BEdKzQ.pgp
Description: PGP signature


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux