This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Memory leak patch-I

From: Uttam Pawar <uttamp at us dot ibm dot com>
To: Andrew Pinski <pinskia at physics dot uc dot edu>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Mon, 20 Feb 2006 10:11:45 -0800
Subject: Re: Memory leak patch-I
References: <1140223686.7074.161.camel@dyn9047018077.beaverton.ibm.com> <d9c2c7ec1ea50c6bd25123e4585548ac@physics.uc.edu>

Hi Andrew,

Thanks for reviewing the patch.

On Fri, 2006-02-17 at 19:48 -0500, Andrew Pinski wrote:
> > +       if (p_false)
> > +         free (p_false);
> 
> You don't need if (p_false) here as free with a NULL argument will
> just
> ignore it.

Should I modify the patch and re-submit the whole patch?

Thanks,

Uttam

Follow-Ups:
- Re: Memory leak patch-I
  - From: Uttam Pawar

References:
- Memory leak patch-I
  - From: Uttam Pawar
- Re: Memory leak patch-I
  - From: Andrew Pinski

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]