Re: src/ui/frmQuery.cpp : tentative patch - Mailing list pgadmin-hackers

From Dave Page
Subject Re: src/ui/frmQuery.cpp : tentative patch
Date
Msg-id E7F85A1B5FF8D44C8A1AF6885BC9A0E4307383@ratbert.vale-housing.co.uk
Whole thread Raw
In response to src/ui/frmQuery.cpp : tentative patch  (Ian Barwick <barwick@gmail.com>)
List pgadmin-hackers

> -----Original Message-----
> From: pgadmin-hackers-owner@postgresql.org
> [mailto:pgadmin-hackers-owner@postgresql.org] On Behalf Of Ian Barwick
> Sent: 24 November 2004 21:45
> To: pgadmin-hackers@postgresql.org
> Subject: [pgadmin-hackers] src/ui/frmQuery.cpp : tentative patch
>
> Hi
>
> when starting pgAdmin with a fresh .pgadmin file, in the
> query tool's "File" menu the option "Recent files" is active,
> but selecting it produces a miniscule box with no content.
> It's not a particularly dramatic problem, but it caused me to
> wonder what recent files it could possibly refer to as I'd
> never started pgadmin on this particular system before.
>
> I'd like to believe the attached patch solves this problem by
> only activating the menu item if at least one recently saved
> file is known. It seems to work.
>
> However I must point out I had only vaguely heard of
> wxWidgets before yesterday and my C++ hasn't been out and
> about much recently, so I wouldn't be surprised if there's a
> better / safer way of doing this. (If there is, and someone
> could point it out, I'd appreciate it, because I'd be
> interested in getting to know the pgadmin internals  a bit better).

Sorry for the delay... Modified patch applied.

Implicitly casting a uint to bool gives a warning in VC++, so I changed
the test to be more precise:

fileMenu->Enable(MNU_RECENT, (recentFileMenu->GetMenuItemCount() > 0));

Thanks for the patch.

Regards, Dave

pgadmin-hackers by date:

Previous
From: cvs@cvs.pgadmin.org
Date:
Subject: CVS Commit by dpage: Disable the recently used files list if there are
Next
From: Andreas Pflug
Date:
Subject: Going for Gold