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
>>> 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
>>> 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.
In response to
pgadmin-hackers by date
|Next:||From: pgAdmin Trac||Date: 2010-08-26 20:33:40|
|Subject: Re: [pgAdmin III] #223: PgAdmin Crash|
|Previous:||From: Guillaume Lelarge||Date: 2010-08-26 20:32:16|
|Subject: pgAdmin III commit: Add to the main window a link to each query window|