Infinite Loop with GetAsyncKeyState

Heres my code:

Code:

if ( GetAsyncKeyState ( VK_LBUTTON ) )
        {

            for ( int i = 0; i < 3; i++ )
            {
                Sleep ( 30 );
                
                
                if ( macro.mouseMacro == 1 )
                {   
                    left_click(); //function i created to simulate left click
                }
                else
                {
                    sk.SendKeys(macro.keyboardMacro.c_str());
                } // end else
            } // end for
        } // end if

For some reason, when I click once, it goes in an infinite loop and keeps simulating a left click, when its actually only supposed to do it only 3 times.

Left Click Function

Code:

int left_click()
{
    // left mouse button down
    INPUT    Input={0};
    Input.type      = INPUT_MOUSE;
    Input.mi.dwFlags  = MOUSEEVENTF_LEFTDOWN;
    ::SendInput(1,&Input,sizeof(INPUT));
    
    // left mouse button up
    ::ZeroMemory(&Input,sizeof(INPUT));
    Input.type      = INPUT_MOUSE;
    Input.mi.dwFlags  = MOUSEEVENTF_LEFTUP;
    ::SendInput(1,&Input,sizeof(INPUT));
}

any help will be appreciated.

thanks,

guitarist809

If you are using this to simulate clicks on your own application, then there could be a problem. When you simulate a left click, the message pump will pick it up and send a WM_LBUTTONDOWN message to that program. if you are sending this simulated key to your own program it will cause an infinite loop.

Also, I noticed in a few cases in my own programs, that some windows API calls will invoke messages to your program, and your program will act upon them. This can cause an infinite loop. You can try putting a break point before the SendInput() call. Once you break on that breakpoint, put another breakpoint in your message handing procedure to see if you are receiving any messages before SendInput() ends.

Its worth a shot, since your logic is correct.