On Tue, Mar 30, 2010 at 4:34 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> This patch attempts to address the issue that at least I've heard a
> couple of time, of that the "execute to file" feature doesn't prompt
> for the filename until after it's finished. This is quite annoying
> when running multi-hour queries.
> Other than a review please, I couldn't find where we actually destroy
> the QueryExecInfo object allocated in frmQuery::execQuery(). It gets
> picked up in OnQueryComplete(), but I don't see a delete anywhere. Is
> it automatically deleted by the wx stuff (in which case I have a leak
> of qi->toFileExportForm that I have to take care of), or do we have a
> leak there today (in which case I have that leak to take care of :D).
It sure looks like a leak to me. Seems like we should delete it after
the call to completeQuery. Not sure if that will leave a leak if the
query is aborted though.
EnterpriseDB UK: http://www.enterprisedb.com
In response to
pgadmin-hackers by date
|Next:||From: Guillaume Lelarge||Date: 2010-04-01 22:54:29|
|Subject: Re: Rework of the frmOptions window|
|Previous:||From: Magnus Hagander||Date: 2010-03-30 15:34:59|
|Subject: Query tool execute to file prompting|