It does indeed work but seems kinda odd in a few ways.

1. Program compiles fine without winable.h and stdio.h, i guess you left them here after trying something.
2. You left in target, pt, wrect in the main function but they never get used. -Wall does not like that so i had to remove those.
3. Those while loops that trap the input are weird on the first look but work better than i would have expected. I tried to find another way but failed although i think there are better ways to do a toggable button.
4. Why do you need pthreads i don't see for what it is needed. Maybe with mingw it is always needed and i just never noticed :P

Those are just tiny things and it does work for such a simple program. It comes out pretty small as well with just 17.4kb (64-bit) and 12.0kb (32-bit). A nice base to build upon to if you like to fiddle around a bit. Anyways, here is the cleaned up version i did compile from:
 

http://pastebin.com/cSCv3jfW

g++ -o clicker.exe main.cpp -Wall -s -O3

Yeah leaving your code lying around doing nothing can be good. I just "love" to come back to old code, not understanding and having to rewrite it :P

I do not use Codeblocks or Eclipse but simply Notepad++. I used Codeblocks in the beginning but as i wanted to use makefiles for compiling it kinda sucked at that. I generally don't like those IDEs so much as you have to create a new project for a simple hello world program. Within my makefiles i always try to use -Wall for showing almost all errors that can and should be fixed even if the program would work with those mistakes. Notepad++ does not show me anything at all.

Not sure about the pthreads as it does not mention to have the sleep function in there. It is in unistd.h though. In the case of this little program it is the wrong function anyways. You are using the WinAPI Sleep() and not the POSIX sleep() but they do pretty much the same stuff. As this is nonportable anyways the program would not benefit from using the POSIX version.

I am an amateur as well doing programming just for fun.

Really cool, those should be combined into one tool that can do both. I tried that with the clicker in a way that it supports toggled and untoggled mode. Here is the code: http://pastebin.com/6cyPgLRf

You can then run it like so: clicker.exe -k a -s 1 -t

I use the normal main instead of WinMain as it is weird to work with command line options in the windows version. Fetching the arguments is done with getopt which is really easy to use. -k sets the key, -s the sleeping time and if -t is set the program runs in toggle mode. This isn't quite optimal but works for now. -k just works with letters and numbers for now, maybe it should accept keycodes.

You are right with using printf for simple error hunting, i also do that. I have literally no experience with debuggers so i have to rely on that and the compiler. Now its in there anyways as i am printing the setup info at the very beginning.

And by the way i am using C++ here while this is still technically pure C here if you change the headers.

OK, this way of trapping the whole loop for key checks annoyed me more than i would have thought. The problem with those traps is that you could not do that for more than one key and it stops the rest of the program. While trying to create that simple clicker with the main Windows methods i encountered that problem again.

Usually when you create a WinAPI program it runs a window procedure. LRESULT CALLBACK WndProc( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam ) Now looking if msg is WM_KEYUP you can use that to do a toggle. But that just works if the window is active and it does not register input on other programs.

So i was back at trying to use this while trapping again but it would have halted the program. I fiddled around for a while and got to a way that works good enough. You can simply check for a keystate and compare it to its last saved state.

bool down_prev = false;
bool down_now = false;

down_now = KeyIsPressed( key );
if (!down_prev && !down_now) { // not pressed}
if (!down_prev && down_now) { // keydown This is what we need }
if (down_prev && down_now) { // holding }
if (down_prev && !down_now) { // keyup }
down_prev = down_now;

This works for one button but with a buffer (or two) would do work with the whole keyboard and mouse. Now we do not need those traps and can do everything in the main loop. It allows me to use the timer functions of the WinApi and create a GUI version of this.

http://pastebin.com/DXUkJXXD