Re: Small annoyance in pgAdmin 1.10 - Mailing list pgadmin-support

From Guillaume Lelarge
Subject Re: Small annoyance in pgAdmin 1.10
Date
Msg-id 200908102046.10471.guillaume@lelarge.info
Whole thread Raw
In response to Re: Small annoyance in pgAdmin 1.10  (Dave Page <dpage@pgadmin.org>)
Responses Re: Small annoyance in pgAdmin 1.10  (Dave Page <dpage@pgadmin.org>)
List pgadmin-support
Le lundi 10 août 2009 à 15:17:17, Dave Page a écrit :
> On Mon, Aug 10, 2009 at 2:08 PM, Dave Page<dpage@pgadmin.org> wrote:
> > On Mon, Aug 10, 2009 at 2:03 PM, Dave Page<dpage@pgadmin.org> wrote:
> >>> 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.
> >>
> >> Yes, just pass by reference:
> >>
> >> bool generateSQL(const wxString &query);
> >
> > Of course, you might want to leave out the const decoration.
>
> Looking at this a little more closely, why not just add a new function:
>
> int gqbController::getTableCount() { return model->tablesCount(); }
>
> and call that from frmQuery? Assuming it can't already see model....

Why didn't I think about this quite easy solution? no issue with pointer or
reference. So here is the patch.

Thanks Dave and Ashesh :)


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

pgadmin-support by date:

Previous
From: Oliver Keller
Date:
Subject: can't invoke the debugger
Next
From: Guillaume Lelarge
Date:
Subject: Re: Malfunction in dropping database with pgAdmin