From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, Mickael Deloison <mdeloison(at)gmail(dot)com>, Andras Fabian <Fabian(at)atrada(dot)net>, pgadmin-support(at)postgresql(dot)org |
Subject: | Re: BUG: pgScript NOT thread safe!! |
Date: | 2009-06-26 08:57:59 |
Message-ID: | 4A448D97.9040107@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
Looks good.
Working fine on linux and windows :)
Dave Page wrote:
> On Fri, Jun 26, 2009 at 5:50 AM, Ashesh
> Vashi<ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> That's really weird.
>> I tried that patch on windows and found that - the behavior of wxMutex is
>> not as expected. :(
>>
>> On Gtk (Ubuntu 9.04 32bit), the message-box does not adjust to the length of
>> the title message.
>> (Please find the attached screen-shot.)
>>
>> Please find the updated patch, it is working on gtk & windows both.
>> Though it has not been tested on Osx, but I think - it should work.
>
> Per IM discussion, I've removed the mutex altogether. This code all
> gets called in the primary UI thread, so there's actually no
> concurrency issue until the script actually starts, thus we can
> protect it with a simple static bool.
>
> I suspect this is why the mutex didn't behave as expected - it's
> intended to keep different threads out of the protected area.
>
> Tests OK on OSX.
>
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2009-06-26 08:58:58 | Re: BUG: pgScript NOT thread safe!! |
Previous Message | Dave Page | 2009-06-26 08:17:16 | Re: BUG: pgScript NOT thread safe!! |