Hi,

when I scroll very fast, I have the problem that the interrupt handling is too slow. When I scroll in the opposite direction, it takes a long time (up to some seconds) until the camera moves in the opposite direction. Does anyone knows how to solve this issue?

LRESULT CALLBACK WndProc(HWND hwnd, UINT umessage, WPARAM wparam, LPARAM lparam)
{
    switch(umessage)
    {
        // Check if the window is being destroyed.
        case WM_DESTROY:
        {
            PostQuitMessage(0);
            return 0;
        }

        // Check if the window is being closed.
        case WM_CLOSE:
        {
            PostQuitMessage(0);        
            return 0;
        }

        

        // All other messages pass to the message handler in the system class.
        default:
        {
            return ApplicationHandle->MessageHandler(hwnd, umessage, wparam, lparam);
        }
    }
}

LRESULT CALLBACK SystemClass::MessageHandler(HWND hwnd, UINT umsg, WPARAM wparam, LPARAM lparam)
{
    if(umsg == WM_MOUSEWHEEL)
        {
            //MessageBox(m_hwnd, L"Mouse wheel", L"Test", MB_OK);
            

            short x = (short)HIWORD(wparam);
            m_Application->HandleInterrupt(MOUSEWHEEL_ROTATED,x);
 
        }
    return DefWindowProc(hwnd, umsg, wparam, lparam);
}

void ApplicationClass::HandleInterrupt(int buttonID, int param1){
    //WaitForSingleObject(InterruptMutex,INFINITE);
    if(buttonID==MOUSEWHEEL_ROTATED){
        if(param1<0){
            targetCameraDistance+=0.5f;
            if(targetCameraDistance>=m_settings->maxCameraDistance){
            targetCameraDistance=m_settings->maxCameraDistance;
            }
        }
        if(param1>0){
            targetCameraDistance-=0.5f;
            if(targetCameraDistance<=0){
            targetCameraDistance=0;
            }
        }
    }
    //ReleaseMutex(InterruptMutex);
}

Do I need the Mutex when handling interrupts?

Greetings,

Magogan

I noticed a lot of people make mistake of using if(PeekMessage(&message, 0, 0, 0, PM_REMOVE)) instead of while(...) inside game loop, this might be source of the problem.

No, I just used the WndProc function, but it seems that it is too slow for my purposes. PeekMessage seems to be fast enough. Now it works :D

Magogan, post the code of your message loop.

That's probably where the problem lies. (As others have pointed out already)

I have fixed the message loop and now it should work.

void SystemClass::Run()
{
    MSG msg;
    bool done, result;


    ZeroMemory(&msg, sizeof(MSG));
    
    done = false;
    while(!done)
    {
        if(PeekMessage(&msg, NULL, 0, 0, PM_REMOVE))
        {
            TranslateMessage(&msg);
            DispatchMessage(&msg);
            
            if(msg.message == WM_QUIT){
                done = true;
            }
            else if(msg.message == WM_MOUSEWHEEL){
                short x = (short)HIWORD(msg.wParam);
                m_Application->HandleInterrupt(MOUSEWHEEL_ROTATED,x);
            }
        }
        else
        {
            
            result = Frame();
            if(!result)
            {
                done = true;
            }
        }

    }

    return;
}

Btw., you are handling WM_CLOSE wrongly. Instead of calling PostQuitMessage you should call DestroyWindow (which then causes WM_DESTROY message and some other things for cleanup). http://msdn.microsoft.com/en-us/library/windows/desktop/ms632617%28v=vs.85%29.aspx