Re: Lightspeed for frmQuery and other issues.

Dave Page wrote:
>
> -----Original Message-----
> From: "Andreas Pflug"<pgadmin(at)pse-consulting(dot)de>
> Sent: 30/04/06 16:19:51
> To: "Dave Page"<dpage(at)vale-housing(dot)co(dot)uk>
> Cc: "pgadmin-hackers(at)postgresql(dot)org"<pgadmin-hackers(at)postgresql(dot)org>
> Subject: Re: Lightspeed for frmQuery and other issues.
>
>>It wasn't removed explicitely, but the underlying class that didn't meet
>>the requirement was backed out. You've painted the wall before
>>wallpapering it.
>
>
> Quickreport sat over that class as well - is that now broken too?

That's in frmQuery only, right? If so, the local solution is fine. If
it's more global, it should go somehow to dlgClasses.
>
> Please restore the functionality or I will back out the patch until it is completed in it's entirety.

Which functionality?
>
> You complain about the work that led to significant speed increase from the original code, but at least we busted a gut to make sure it didn't break any existing functionality.

Again: I said early enough the base was wrong. And I didn't see *any*
speed increase on linux. Probably, any virtual solution will be ok, but
any non-virtual solution *cant* be ok.

>
>
>>It's not a problem of the factories, they do what they should and will
>>work *if used*.
>
>
> Like I said, I'll look at this. I didn't grok that the failed attempt you mentioned was only a minor rejig.

It should have been, but things are horribly intervowen. I'd have to
understand in detail what's going on to continue, because apparently
removing MNU_XXX entries would kill methods too.

Regards,
Andreas