Re: Small annoyance in pgAdmin 1.10

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: Guillaume Lelarge <guillaume(at)lelarge(dot)info>, 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:17:17
Message-ID: 937d27e10908100617l67d9e990s706331f1f45bd2f4@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-support

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....

--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com

In response to

Responses

Browse pgadmin-support by date

  From Date Subject
Next Message Luiz K. Matsumura 2009-08-10 13:34:39 Feature request - View Filtered rows
Previous Message Dave Page 2009-08-10 13:08:44 Re: Small annoyance in pgAdmin 1.10