From: | Andreas Pflug <pgadmin(at)pse-consulting(dot)de> |
---|---|
To: | Dave Page <dpage(at)vale-housing(dot)co(dot)uk> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Jerry LeVan <jerry(dot)levan(at)eku(dot)edu> |
Subject: | Re: 1.4.1 On OSX |
Date: | 2005-12-21 14:02:17 |
Message-ID: | 43A96069.9040902@pse-consulting.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Dave Page wrote:
>
>
> On 21/12/05 1:12 pm, "Andreas Pflug" <pgadmin(at)pse-consulting(dot)de> wrote:
>
>
>>Dave Page wrote:
>>
>>>Hi Andreas,
>>>
>>>I finally tracked down the bug in 1.4.1 on Mac which causes a crash whenever
>>>pretty much any property dialogue is opened. Basically, in
>>>ctlComboBox::GetSelection() we call GetCurrentSelection(). In wxMac however,
>>>this simply calls GetSelection() which is a virtual and results in our
>>>GetSelection() actually getting called, in turn causing a loop and eventual
>>>stack overflow.
>>>
>>>So, the attached patch seems to fix the problem and return the auto-complete
>>>code to the same slightly-broken-but-usable state that it was in 1.4.0 (i.e,
>>>you can only type the first character of a data type for example, and the
>>>first match will be selected).
>>>
>>>Can you take a look and confirm my analysis (or call me an idiot, whatever
>>>is correct!) before I apply this to both branches and packare 1.4.1.plus
>>>please?
>>
>>In general I agree. I committed a fix in ctlComboBoxFix doing
>>essentially the same, but the version dependent GetCurrentSelection code
>>isn't cluttered around the sources this way.
>
>
> Annoyingly that's one of the first fixes I tried and it didn't work.
Well the wxComboBox code is a shy guy (remember how reluctant it's
mentioned in 2.6.2 release notes). Maybe you pressed a keyboard button
too harsh, and it was frightened :-)
Regards,
Andreas
From | Date | Subject | |
---|---|---|---|
Next Message | svn | 2005-12-21 14:10:30 | SVN Commit by dpage: r4851 - branches/REL-1_4_0_PATCHES/pgadmin3/src/include/ctl |
Previous Message | Dave Page | 2005-12-21 13:48:31 | Re: 1.4.1 On OSX |