From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-support(at)postgresql(dot)org, Magnus Hagander <magnus(at)hagander(dot)net>, Rainer Bauer <usenet(at)munnin(dot)com> |
Subject: | Re: Small annoyance in pgAdmin 1.10 |
Date: | 2009-08-10 13:02:19 |
Message-ID: | 4A801A5B.1040208@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
Hi Guillaume,
Guillaume Lelarge wrote:
> Le lundi 10 août 2009 à 14:49:21, Dave Page a écrit :
>> On Mon, Aug 10, 2009 at 1:44 PM, Guillaume
>>
>> Lelarge<guillaume(at)lelarge(dot)info> wrote:
>>> Here is a patch that should fix this. Magnus, care to check it? I'm not
>>> ready to commit it without proper review. It plays with pointers, and
>>> that scares me a bit :)
>> Passing a pointer to a wxString around is *always* wrong. It can futz
>> with the internal reference counting.
> If I can't use a pointer, I don't see how I can do this. Or should I just
> remove the MessageBox?
As per my understanding, you always can use reference in place of pointer.
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2009-08-10 13:03:42 | Re: Small annoyance in pgAdmin 1.10 |
Previous Message | Guillaume Lelarge | 2009-08-10 13:00:00 | Re: Small annoyance in pgAdmin 1.10 |