what do you think of my message loop for DirectX proj

[tom_dx]

MSG msg; while( TRUE ) { if ( PeekMessage( &msg, NULL, 0, 0, PM_REMOVE ) ) { if ( msg.message == WM_QUIT ) { break; } TranslateMessage( &msg ); DispatchMessage( &msg ); } } IM PROUD TO BE A GMAIL;

[Cedric Moonen]

Yes, it's a message loop and ...??? What is EXACTLY the question ?

[tom_dx]

do you think this would be substantially enough for a DX game? IM PROUD TO BE A GMAIL;

[Stephen Hewitt]

Because you're using PeekMessage rather then GetMessage your loop spins in the outer while loop even when there is no message to process. Unless you have an else not show in the code above in the if ( PeekMessage( &msg, NULL, 0, 0, PM_REMOVE ) ) statement this code is needlessly inefficient. I would suggest you replace it with a GetMessage and watch the CPU usage go down. Steve

[tom_dx]

thank you for the comment, and yes I do have an else statement in the block but forgot to write it in. so you think it will still be good if it has an else block or should i stick with getmessage? IM PROUD TO BE A GMAIL;

[Stephen Hewitt]

Normally you'd use PeekMessage and perform some idle time processing in the else part. I'm not sure what's going on in the else bit but it looks ok to me. I find myself a little concerned that you may be busy waiting but I'd have to see the else bit to be sure. Steve