views:

232

answers:

2

Hi All!

I use SetTimer API and I see a lot of code like this:

case WM_DESTROY: 
    // Destroy the timer. 
    KillTimer(hwnd, IDT_TIMER); 
    PostQuitMessage(0); 
    break;

Do I have to call KillTimer or the system will automatically free resources on the process exit? Does forgetting to call KillTimer lead to resource leaks?

I understand that if the timer is not needed it CAN be destroyed by KillTimer. But MUST it be destroyed manually?

+3  A: 

The timer will be destroyed automatically by Windows on process exit.

But bear in mind that (so it appears) your timer belongs to the window, not the process. So if your application allows these windows to be created and destroyed within a process, you'll be leaking timers.

It's always good practice to clean things up explicitly, because otherwise the lack of cleanup can come back to bite you later on.

RichieHindle
I don't believe that is correct to say timers will leak when the window is destroyed. Maybe in the old Windows 95/98 days, a timer could leak. But definitely on XP and up, timers set on hwnds are cleaned up when the window is destroyed.
selbie
@selbie: Yes, I'm sure you're right. But I still say it's good practice to assume that things will leak unless you clean them up explicitly. Imagine you change from window-based timers to callback-based ones - then you will have a leak unless you explicitly kill them.
RichieHindle
+2  A: 

Timers set from HWNDs are implicitly destroyed by the window (hwnd) being destroyed. So no, you don't have to clean up your timers when the window exits.

But it's a good practice to have all your resources related to the window cleaned up on window close.

selbie