Re: [Gimp-developer] [PATCH] Fix some memleaks

2011-08-18 Thread Kevin Cozens
On 11-08-17 01:16 PM, Nelson A. de Oliveira wrote:
> Updated patch available at
> http://people.debian.org/~naoliv/misc/0001-Fix-some-memleaks-2.txt

Patch was committed as revision ce3db28f5abe3667a97be3ad2d44a0794efecb16. 
Thanks again for spotting the leaks and the patch.
___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer


Re: [Gimp-developer] [PATCH] Fix some memleaks

2011-08-17 Thread Nelson A. de Oliveira
Hi!

On Wed, Aug 17, 2011 at 1:43 PM, Kevin Cozens  wrote:
> After you make the above mentioned changes, send a link to the updated patch
> file.

Updated patch available at
http://people.debian.org/~naoliv/misc/0001-Fix-some-memleaks-2.txt

Thank you!

Best regards,
Nelson
___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer


Re: [Gimp-developer] [PATCH] Fix some memleaks

2011-08-17 Thread Kevin Cozens
On 11-08-16 07:14 PM, Nelson A. de Oliveira wrote:
> Can somebody verify and commit it, please?
> http://people.debian.org/~naoliv/misc/0001-Fix-some-memleaks.txt

Thank you for the patch. There are some minor changes needed before the 
patch can be applied.

The patch file should not contain tabs. Please change the tabs before the 
g_free() calls you added to spaces.

The second change to psd-util.c frees pascal_str but it looks to me that it 
also needs to be freed after the call to g_debug().

After you make the above mentioned changes, send a link to the updated patch 
file.
___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer