From: | Jasmin Dizdarevic <jasmin(dot)dizdarevic(at)gmail(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Discussion - Search Objects |
Date: | 2011-06-16 20:31:09 |
Message-ID: | BANLkTi=HyL6nW6+LR2Ye6j+xGvS4XZwM7w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
2011/6/16 Guillaume Lelarge <guillaume(at)lelarge(dot)info>
> On Wed, 2011-06-15 at 23:48 +0200, Jasmin Dizdarevic wrote:
> > Yeah, I know. All of the dialogs have a "binding" to some pgObject,
> > because all of them modifies something.
>
> No, see dlgFindAndReplace.
>
> > When I started working I had no clear picture about the class
> > hierarchy and I thought this would be an easier entry point. I can
> > change the prefix if this is wished.
> >
>
> Yeah, I think it would be better.
>
>
I'm going to change that.
>
> > btw: You could help me with the factory class. Am I starting the
> > dialog correctly?
> >
>
> Seems good to me. My only question is: why in frmMain.ccp and not in
>> frmSearchObject.cpp? I first tried the latter, and was surprised to find
>> it in the former.
>
>
> I'm currently surprised myself ;)
> BTW, can you stop top-posting please? Thanks.
>> Regards.
>
>
> I'll try it.
One more thing:
Can somebody say me how to merge different patches to a single one? I've
made 3-4 commits and would like to have exactly one patch file. It's not so
important, but it would be very nice.
Generally, I wanna thank you for your patience and help - I know this can be
frustrating to answer questions, that are clear to you since a long time :)
> --
> Guillaume
> http://blog.guillaume.lelarge.info
> http://www.dalibo.com
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2011-06-16 20:39:02 | Re: Discussion - Search Objects |
Previous Message | Dave Page | 2011-06-16 20:06:25 | Re: Discussion - Search Objects |