On 09.09.2015 08:34, Juha Manninen wrote:
On Wed, Sep 9, 2015 at 8:09 AM, patspiper <[email protected]> wrote:
The patch works indeed. I haven't inspected the dektop manager code, but
QueueAsync call may be more suited for the task than a timer.
Another choice in OnIdle handler.
The benefit would be that also the initial update of the IDE Coolbar
would be delayed and the IDE would be ready for editing sooner.
Lazarus still starts a lot slower than many competing advanced
programmers' editors. There is no good reason for it because we don't
use virtual machines like Java or .NET based systems do (including
Delphi now).
I optimized the Component Palette startup a lot but there is still more to do.

Juha
Yes, the TTimer was just a quick&dirty solution to see if such a workaround helps. I admit that it wasn't a good choice. The desktop name has to passed over to the function so QueueAsync looks OK for me.

But still, it doesn't change anything about the launch time of Lazarus. Only the toolbar button click has to be decoupled from the message queue. Nothing else.

Another question is if this should be handled in the LCL already and allow TToolButton be destroyed in MouseDown / from drop down menu? I see that rev 49465 changed the code but the old code didn't allow it either.

Ondrej

--
_______________________________________________
Lazarus mailing list
[email protected]
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to