On Mon, Mar 23, 2015 at 05:37:32PM +0200, Claudiu Olteanu wrote:
> Yes, you are right. There is a simple white space fix in the
> context of the memory leaks. I thought that since it is regarded
> with the memory allocation/deallocation it wouldn't be a problem and
> a second patch will be unnecessary. Sorry, my mistake.
> I will create a new one when I will get home.
> 
> I also looked over the patch sent by Salvador and I believe
> that there will be no problem if we will merge this two patches.
> He doesn't use the freed variables and the ones implied are deallocated
> before a return statement.

Well, the patches conflict.

I'm OK with a single whitespace fix in the middle.
But I'd like to see new clean patches

/D
_______________________________________________
subsurface mailing list
subsurface@subsurface-divelog.org
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to