Re: Ticket #223

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:03:06
Message-ID: 4C76C87A.3020607@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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 :)

--
Guillaume
http://www.postgresql.fr
http://dalibo.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2010-08-26 20:32:16 pgAdmin III commit: Add to the main window a link to each query window
Previous Message Guillaume Lelarge 2010-08-26 20:00:06 Re: 1.12 RC for monday - translations needed