Skip site navigation (1) Skip section navigation (2)

Re: Enabling SQL text field in the SQL tab of object dialog

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)postgresql(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Enabling SQL text field in the SQL tab of object dialog
Date: 2008-08-26 21:50:14
Message-ID: 48B47A96.5000708@lelarge.info (view raw or flat)
Thread:
Lists: pgadmin-hackers
Dave Page a écrit :
> On Tue, Aug 26, 2008 at 6:48 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> Guillaume Lelarge a écrit :
>>> [...]
>>> Back on this. The plan took a lot more time than I initially thought.
>>> Anyways, here is the v5 patch updated.
>>>
>> I'm tired for sure... here is the patch.
> 
> :-(
> 
> Well it looks good (you've *really* got the hang of XRC/wxSizers now
> :-) ) - but if I open the 'New Database' dialogue, enter a name, and
> then click on the SQL tab, I get an assert on OSX.
> 

This issue has nothing to do with this patch. I think this bug comes
from my previous commit :-/

Without explicitely setting cbTemplate selection, cbTemplate->GetValue()
 gets out with a failed assert. AFAICS, my previous patch didn't modify
this... but I find it weird that no one hits this bug before.

Anyways, the good news is that it gave me an idea to make dlgLanguage
growable. I just commited this dialog. I also commited the dlgDatabase fix.

> [...]
> I also get a crash if I try to open the New Table dialogue. I suspect
> that is the possible incompatibility that you were worried about,
> though I guess it could be related to this patch.
> 

I'm sorry, I don't have this issue. Surprising because I don't see how
the previous fix could have fixed it.

No new patch, because the fix is already commited.


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

In response to

Responses

pgadmin-hackers by date

Next:From: svnDate: 2008-08-26 22:24:52
Subject: SVN Commit by guillaume: r7423 - in trunk/pgadmin3/pgadmin: dlg include/dlg ui
Previous:From: svnDate: 2008-08-26 21:38:26
Subject: SVN Commit by guillaume: r7422 - in trunk/pgadmin3/pgadmin: dlg include/dlg ui

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group