Re: PGA3 Memory Leaks

From: efesar <efesar(at)nmia(dot)com>
To: Andreas Pflug <Andreas(dot)Pflug(at)web(dot)de>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: PGA3 Memory Leaks
Date: 2003-04-16 19:42:20
Message-ID: NGBBKFMOILMAGDABPFEGMEALEEAA.efesar@nmia.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers


> PLEASE undo your undos. The code will CRASH on GTK before the settings
> are written. I didn't go into detail which line is the offending one.
> Memory leaks on program exit is a waste of brain in my opinion. The
> operating system will do all cleanup for us.
> Since there are no trees, menues and so on destroyed and dynamically
> recreated, we don't need to care what's happening to those at the time
> of program end. And at last: I wouldn't have touched that code if
> it worked!

Andreas,

The wxWindows documentation specifically states those objects must be
deleted.

The one line I didn't uncomment was "delete statistics." Since statistics is
a wxListCtrl, it probably doesn't need to be deleted. That's why I left it
commented. Please test and document. It's possible your crash was caused by
that.

In the meantime, I think the change should stay. IMHO, memory leaks are not
wastes of time.

Dave -- Opinion? I'll leave this one up to you, it that's okay.

-Keith

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Andreas Pflug 2003-04-16 21:07:06 Re: PGA3 Memory Leaks
Previous Message Andreas Pflug 2003-04-16 19:16:42 Re: PGA3 Memory Leaks