TCustomForm.ShowModal() implementation have call to Application.Idle() after closing form, for what ??
background: My some forms idle tasks have appeal to form's visual components, so i got AV.
I use standart TApplicationProperties.OnIdle() event. I can, sure, implement closing checking myself in my forms, but, i sure, ShowModal() is erroneous to call
Application.Idle() after form closing.

* Andrey Gusev [Fri, 16 Nov 2007 17:35:26 +0300]:

I can, sure, implement closing checking myself in my forms, but, i

sure,

ShowModal() is erroneous to call Application.Idle() after form closing.

Or, probably, TApplicationProperties component implementation erroneous
- there must form closing checked ?

* Andrey Gusev [Fri, 16 Nov 2007 17:35:26 +0300]:

Can anybody explain to me, why dynamically created forms with TApplicationProperties.OnIdle(),
incline to AV ??? With Delphi it was impeccability ! Please see attached sample. Stack backtrace included.

--------------------
first message

now only: it seems

ShowModal() is erroneous to call Application.Idle() after form closing.

--------
later

Or, probably, TApplicationProperties component implementation

erroneous

- there must form closing checked ?

rather: form existence

Because the destructor was missing. I added it, so the TApplicationProperties now unset the events on destruction.

You misunderstood the code: TCustomForm.ShowModal only calls Idle if *not* closing. See your backtraces: It's the normal Application that calls Idle.

Note: TApplicationProperties should be changed to use the handler lists (e.g. AddOnIdleHandler) instead of the global events. At the moment there can only be one TApplicationProperties at a time.

Mattias

* Mattias Gaertner [Tue, 20 Nov 2007
01:14:27 +0100]:

Thanks for hint, i will try it - on my workproject, but with Delphy it was not neccessarily...

Maybe you misunderstood. It was a bug in the LCL, not in your application.

Mattias

* Mattias Gaertner [Tue, 20 Nov 2007
10:55:49 +0100]:

I understand it now. I must lead this to mantis ?, for your statistic calculations or whatever ?

Only if you opened a bug report for this.

Mattias