[adma]

Originally Posted by Dragokas off. One more question, please: Can I withdraw the TIMER_DATA_HNDL_CLOSE flag in timer callback? Or it is safe to use CloneHandle there: Spoiler PHP Code:         DataPack hPack = new DataPack();         hPack.WriteCell(g_ConVarWitchCount.IntValue);         hPack.WriteFloat(vecPrime[0]);         hPack.WriteFloat(vecPrime[1]);         hPack.WriteFloat(vecPrime[2]);         CreateTimer(GetRandomFloat(g_ConVarDelayMin.FloatValue, g_ConVarDelayMax.FloatValue), Timer_SpawnWitch, hPack, TIMER_DATA_HNDL_CLOSE | TIMER_FLAG_NO_MAPCHANGE); // ... public Action Timer_SpawnWitch(Handle timer, DataPack hPack) {     hPack.Reset();     int iWitches = hPack.ReadCell();     float vecPrime[3];     vecPrime[0] = hPack.ReadFloat();     vecPrime[1] = hPack.ReadFloat();     vecPrime[2] = hPack.ReadFloat();          SetWitch(vecPrime);          iWitches--;          if (iWitches > 0) {         hPack.Reset();         hPack.WriteCell(iWitches);         CreateTimer(GetRandomFloat(g_ConVarIntervalMin.FloatValue, g_ConVarIntervalMax.FloatValue), Timer_SpawnWitch, CloneHandle(hPack), TIMER_DATA_HNDL_CLOSE | TIMER_FLAG_NO_MAPCHANGE);     } }  thanks.

I think what youre doing in that snippet is legal, but why not reuse the same DataPack and close the handle if iWitches == 0? (and remove TIMER_DATA_HNDL_CLOSE)

[Dragokas]

Because it will cause handles leak if timer launched, but had no time to trigger until map end.

[adma]

Forgot about that, so your code makes sense. AFAIK, TIMER_DATA_HNDL_CLOSE just performs a CloseHandle() on the passed data after the timer is stopped/ends. You CloneHandle() the pack handle before timer end, which prevents the underlying pack data from being destroyed. When that iWitches > 0 check fails, the TIMER_DATA_HNDL_CLOSE will close and no more handles will point to the pack data, so then the pack data itself is destroyed.

[Dragokas]

Yes, it is exactly what I am doing in above code