From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | pg(at)kuhnsteffen(dot)de |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Ticket #223 |
Date: | 2010-08-26 20:33:14 |
Message-ID: | 4C76CF8A.9010401@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le 26/08/2010 22:03, Guillaume Lelarge a écrit :
> Le 26/08/2010 21:36, Guillaume Lelarge a écrit :
>> Le 26/08/2010 17:40, Steffen Kuhn a écrit :
>>> Hi Guillaume,
>>>
>>> first of all: you are quite hardworking today.
>>>
>>> I thought about #223 because I really don't like app crashes.
>>> WxWigets tells me that frames opened before app main loop are not
>>> destroyed.
>>> (http://docs.wxwidgets.org/2.8/wx_wxappoverview.html#wxappshutdownoverview)
>>>
>>> So I see there two possibilites:
>>> _1 Say this is feature like wxWidgets does and care about not unchecked
>>> accessing frmMain in frmQuery which is not null but invalid in frmQuery
>>> _2 Close descendant frames on frmMain exit, by either delegate opening
>>> to frmMain so this is done by default or take care about in
>>> frmMain::OnCLose
>>> or pgAdmin3::OnExit
>>> What do you think about this.
>>>
>>
>> I actually don't think it is a good idea to allow a user to use both -q
>> and -s at the same time. If you use -q, it means you don't want to use
>> the browser. But -s launches the browser. Kind of weird, don't you
>> think? so my question is: why would anyone use both of them at the same
>> time? (the same question goes for -S and -s)
>>
>>
>
> BTW, we have the same issue with the frmStatus window. This window can
> launch a frmQuery window, but it won't notify the frmMain window. If you
> open a frmQuery window from the frmStatus one, and then close the
> frmMain window, the frmQuery window will stay open and won't quit. Next
> time you fire some SQL statement or change your connection, kaboom :)
>
>
So you were definitely right with your patch. I added the same kind of
code in the frmStatus window. Should fix #223 all right.
Thanks a lot, Steffen.
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | pgAdmin Trac | 2010-08-26 20:33:40 | Re: [pgAdmin III] #223: PgAdmin Crash |
Previous Message | Guillaume Lelarge | 2010-08-26 20:32:16 | pgAdmin III commit: Add to the main window a link to each query window |