Re: Small annoyance in pgAdmin 1.10

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: 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:00:00
Message-ID: 200908101500.01133.guillaume@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-support

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?

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

In response to

Responses

Browse pgadmin-support by date

  From Date Subject
Next Message Ashesh Vashi 2009-08-10 13:02:19 Re: Small annoyance in pgAdmin 1.10
Previous Message Dave Page 2009-08-10 12:49:21 Re: Small annoyance in pgAdmin 1.10