From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Patch: Crash on linux-x64 |
Date: | 2009-03-11 15:24:13 |
Message-ID: | 937d27e10903110824y36862b41y13ece6de0a392bde@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Wed, Mar 11, 2009 at 10:15 AM, Ashesh Vashi
<ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> Dave Page wrote:
>
> Couple of thoughts about the patch...
>
> We should use #ifdef __WXGTK__ not __LINUX__ as we also support *BSD
> and Solaris, *but*, although the bug only occurs on Linux,
>
> ok
>
> Should we just use this search code on all platforms for consistency and
> future
> enhancement?
>
> I think yes - that will make the behaviour consistent on all the platform.
>
> In wxWidgets, It starts searching the node starts with the pressed character
> from the selected node.
> If no such node found, it start searching the similar node from the ROOT
> node.
>
> In our function (FindItem), we look for the children, grand-children and
> siblings below the selected node as the search criteria.
>
> Don't worry about submitting a new patch - I'll hack up whats there
> based on what we decide.
>
> :)
Thanks, applied for all platforms.
--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | svn | 2009-03-11 16:39:02 | SVN Commit by dpage: r7674 - trunk/pgadmin3/docs/en_US/images |
Previous Message | svn | 2009-03-11 15:23:32 | SVN Commit by dpage: r7673 - in trunk/pgadmin3/pgadmin: ctl include/ctl |