RSS/Atom feed Twitter
Site is read-only, email is disabled
  1. Chat
  2. Useful links
  3. Polls

Featured content

[PATCH] Fix some memleaks

This discussion is connected to the gimp-developer-list.gnome.org mailing list which is provided by the GIMP developers and not related to gimpusers.com.

This is a read-only list on gimpusers.com so this discussion thread is read-only, too.

4 of 4 messages available
Toggle history

Please log in to manage your subscriptions.
[PATCH] Fix some memleaks Nelson A. de Oliveira 16 Aug 23:14
  [PATCH] Fix some memleaks Kevin Cozens 17 Aug 16:43
   [PATCH] Fix some memleaks Nelson A. de Oliveira 17 Aug 17:16
    [PATCH] Fix some memleaks Kevin Cozens 18 Aug 20:43
Nelson A. de Oliveira
2011-08-16 23:14:03 UTC (over 13 years ago)

[PATCH] Fix some memleaks

Hi!

Can somebody verify and commit it, please? http://people.debian.org/~naoliv/misc/0001-Fix-some-memleaks.txt

Thank you!

Best regards, Nelson

Kevin Cozens
2011-08-17 16:43:13 UTC (over 13 years ago)

[PATCH] Fix some memleaks

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.

Nelson A. de Oliveira
2011-08-17 17:16:33 UTC (over 13 years ago)

[PATCH] Fix some memleaks

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

Kevin Cozens
2011-08-18 20:43:25 UTC (over 13 years ago)

[PATCH] Fix some memleaks

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.