From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Query tool execute to file prompting |
Date: | 2010-04-06 16:55:42 |
Message-ID: | p2s9837222c1004060955q4d7aa72encafc04f584e5f9da@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Wed, Mar 31, 2010 at 12:01 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> On Tue, Mar 30, 2010 at 4:34 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> Hi!
>>
>> 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.
>
> Looks sane.
>
>> 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.
Doesn't that just say that we have to delete it in OnCancel() as well?
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2010-04-06 17:04:53 | Re: Query tool execute to file prompting |
Previous Message | Ashesh Vashi | 2010-04-05 11:55:35 | Re: PATCH: Ticket#99: Support for Default Privileges |