| From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
|---|---|
| To: | Dave Page <dpage(at)postgresql(dot)org> |
| Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: Beta 2 Wednesday? |
| Date: | 2007-07-30 08:45:55 |
| Message-ID: | 46ADA543.40909@lelarge.info |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
Dave Page a écrit :
> Guillaume Lelarge wrote:
>[...]
>> OK, I think I've found a fix to Pavel's issue. To check cbLanguage, we
>> use GetCurrentSelection. When I try to change an already defined
>> function, GetCurrentSelection always returns -1 which is an invalid
>> value. If I use GetGuessedSelection (with every other changes needed to
>> use ctlComboBox instead of wxComboBox), it works great. I'm not sure
>> this is the right way to fix this, so here is the patch.
>>
>> As always, comments welcomed.
>
> Hmm, sounds like it might be platform specific issue, similar to (but
> not the same) one Stefan fixed for me in wxMac before 2.8 was released.
>
> What OS and version of wxWidgets is this?
>
Kubuntu 7.04 (linux 2.6.20).
wxWidgets 2.8.4 (kubuntu's package).
I don't know for Pavel.
> I have no problem with the patch, so if this is reproducable with the
> latest wx version, please go ahead and apply.
>
If you're still OK, I will commit it today.
Regards.
--
Guillaume.
<!-- http://abs.traduc.org/
http://lfs.traduc.org/
http://docs.postgresqlfr.org/ -->
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dave Page | 2007-07-30 08:57:52 | Re: Beta 2 Wednesday? |
| Previous Message | Dave Page | 2007-07-30 08:39:00 | Re: Beta 2 Wednesday? |