Okay, it's broken. Theres a lot of code here! Please help

It's basically the one from opengl-tutorial.com. Using glm but not glfw.

The camera kind of works but doesn't.. it's jerky and when I go left with the mouse it goes and looks up etc.. when I go up it goes down..

all sorts of crazy

If we can fix this maybe we can fix the keyboard input :/

void UpdateFromMouse( int w, int h, int mousex, int mousey )
		{	
			/*
				Handle the mouse input
			*/
			_mx = mousex;		_my = mousey;

			m_horizontalAngle += m_mouseSpeed * m_frameTime * float( w / 2 - _mx );
			m_verticalAngle   += m_mouseSpeed * m_frameTime * float( h / 2 - _my );

			m_direction = glm::vec3
			(
				cos( m_verticalAngle ) * sin( m_horizontalAngle ),
				sin( m_verticalAngle ),
				cos( m_verticalAngle ) * cos( m_horizontalAngle )
			);

			m_right = glm::vec3
			(
				sin( m_horizontalAngle - 3.14f / 2.0f ),
				0,
				cos( m_horizontalAngle - 3.14f / 2.0f )
			);

			m_up = glm::cross( m_right, m_direction );

			/*
				Update the view matrix
			*/
			m_viewMatrix = glm::lookAt( position, position + m_direction, m_up );
		}

Without looking at the original code, I am pretty sure that you should go back and have a very close look at what they are doing with the mouse coordinates. Limited to the code above it is completely pointless to store mx and my and you really need to make up your mind about whether you want to use mouse position or mouse movement.

Movement please!

Which code would you like to see? I'm happy to share :)

I´m having the same problem.
Any success?

Trienco:
You´r right, but there is a missing code there (at least on my code)

At the end of code there should be:

glfwSetMousePos(w/2, h/2); 

So the mouse is always on center.

Not using GLFW, using SetCursorPosition and GetCursorPositition from Win32 API.. Soul, I reset the cursor pos outside this call, after I call cam->UpdateFromMouse( x, y, w, h,)

I think I bodged it to work anyways.. trigonometry abuse.

void UpdateFromMouse( int w, int h, int mousex, int mousey )
        {    
            _mx = (float) mousex;        _my = (float) mousey;

            m_frameTime = g_clock->GetFrameTime() ;   ///always returns like 0.000000000001 so hence bodge below..
            
            m_frameTime = 0.01;

            m_horizontalAngle -= m_mouseSpeed * m_frameTime * float( _mx );
            m_verticalAngle     -= m_mouseSpeed * m_frameTime * float( _my );

            m_direction = glm::vec3
            (
                cos( m_verticalAngle ) * sin( m_horizontalAngle ),
                sin( m_verticalAngle ),
                cos( m_verticalAngle ) * cos( m_horizontalAngle )
            );

            m_right = glm::vec3
            (
                sin( m_horizontalAngle - 3.14f / 2.0f ),
                0,
                cos( m_horizontalAngle - 3.14f / 2.0f )
            );

            m_up = glm::cross( m_right, m_direction );

            /*
                Update the view matrix
            */
            m_viewMatrix = glm::lookAt( position, position + m_direction, m_up );
        }

then in 'main'..

 
//att initialisation
mainCam->Create( theClock );
 
//at frame()/ update / message handler::
 
case WM_LBUTTONDOWN: {
            mDown = true;
            ShowCursor(false);

            SetCapture(hWnd);                                    // capture the mouse.      
            GetCursorPos(&cPos);                                // store cursor position
            SetCursorPos(screenWidth / 2, screenHeight / 2);    // reset to some default position
                
            return 0;
        }
            
        case WM_LBUTTONUP: {
            if ( mDown ) {
                mDown = false;    
        
                SetCursorPos( cPos.x, cPos.y );                    // set the cursor to its position
                ReleaseCapture();                                // release the capture                        
            }
            ShowCursor( true );                                    // show it
            
            return 0;
        }
            
        case WM_MOUSEMOVE: {
            if( mDown ) {
                GetCursorPos( &currentPos ); // screen coords

                disp.x = currentPos.x -  screenWidth / 2;
                disp.y = currentPos.y -  screenHeight / 2;

                /*    Update the Camera     */
                m_GFX->mainCam->UpdateFromMouse( screenWidth, screenHeight, disp.x, disp.y );

                SetCursorPos( screenWidth / 2, screenHeight / 2 );
                return 0;
            }
        }