TRtcTimer cleanup problem

After the last update my server does not stop as nicely as before. I have a situation where i want to .Stop(..) a timer in the "ProviderModule" destructor. Since the destructor is run on application termination (and not on server stop) it will be executed *after* the background management thread has ended resulting in an AV.

To reproduce using the "RTCWebServer" Demo:

1. Open the rtcFileProvider.pas unit.
2. Add a private "TM: TRtcTimer;" and add a private "procedure DummyEventHandler;".
3. In DataModuleCreate, add these lines before "end":
TM := TRtcTimer.Create(true);
TRtcTimer.Enable(TM, 30 * 1000, DummyEventHandler);
4. In DataModuleDestroy, add "TRtcTimer.Stop(TM);" after "File_Provider := nil;".
5. Fix uses clause and give DummyEventHandler an empty implementation.

Now if you start the server explicitly, access it, the stop it and after it has stopped close the application you'll get the same AV as me: in TRtcTimerThread.RemoveTrigger @"FCS.Acquire;". (Seems it can happen in other places too).

I'm not sure this is the best way of doing things. I could probably move the creation, start and destruction to where i handle server start and stop and the problem will most likely go away. I'm posting this so that i can be clear about if this was unintended behaviour of if i was "lucky" when i first wrote that code.

Regards,

/Dany