Re: Small annoyance in pgAdmin 1.10

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(at)pgadmin(dot)org> wrote:
> > On Mon, Aug 10, 2009 at 2:03 PM, Dave Page<dpage(at)pgadmin(dot)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.org
http://dalibo.com