Hey everyone,
So, when someone right clicks with the mouse, I display an image. When they right click again, the image should disappear. Here's a big chunk of what I have to accomplish this:

So, when I run this and right-click over and over, most of the time, the image appears where I've right clicked. Then a few times it disappears. Remember, goal is to right-click 1: image appears; right-click2: image disappears, and so on.

So what obvious thing am I messing up with?

From first glances: at the end of your "else if(mouse_b & 2)" section, put a "while (mouse_b & 2);". You're not waiting for the mouse button to go up, and so the code is repeatedly run while you press the right mouse button.

Jeroen

[edit]
However, if you want the rest of the program to keep executing. In which case, try:

Sorry, at work at the moment, so I can't determine if the code is 100% correct.

Hehehe. Ta for the correction. My C++ syntax is still lacking.

Jeroen

hey again both of you!
sorry I haven't replied, busy with my other coures as well.

Of course, your explanation makes sense. I'm trying to make changes like you've stated. I'm declaring bool rmb_down = false right after I do exist so that it is global.

However, I keep getting this stubborn errors:
19 C:\ind study\Project 1\main.c syntax error before "rmb_down"
19 C:\ind study\Project 1\main.c `false' undeclared here (not in a function)

Why does it complain about using a bool and false..?

P.S. I think I tried to use exist as a bool and I was getting same errors.
P.S.S I am using rmb_down as an int and using 0 and 1 for false and true. So yeah, thanks, now it works as it's suppose to.

Could you post your new code?

You are compiling as C++, and not just C, right?

Lenny,
I believe it is C.

Bool is a C++ only type.

You can implement a similar type in C using enumerations though.

typedef enum { FALSE, TRUE } bool;

Actually, C99 has bool, amongst other types that were formerly C++ only. Not that his compiler seems to support C99.

int draw_bitmap()
{
    BITMAP *ship = load_bitmap("spaceship.bmp", NULL);                
    mx = mouse_x;
    my = mouse_y;
    masked_blit(ship, screen, 0, 0, mx, my, 64, 64);
    
}

I think this might keep causing your program to take up more and more memory. You only have to load the bitmap once, not every time you use it. Once you are done with it you have to free it.

Don't forget to delete your pointer have you're done with it... Which, if you were going to keep doing it this way, then you'd need

int draw_bitmap()
{
    BITMAP *ship = load_bitmap("spaceship.bmp", NULL);                
    mx = mouse_x;
    my = mouse_y;
    masked_blit(ship, screen, 0, 0, mx, my, 64, 64);
    destroy_bitmap(ship);
}

Otherwise destroy_bitmap() at the end of your program. =]

hey guys,
thanks for catching that I wasn't deleting the bitmap. I made that change. One weird thing though:

when I right-click, all is well. say I just right-click 3 times and the image appears on screen, then left-click. this gets rid of the image (which it shouldn't!). then if you right-click, it doesn't bring up the image till you right-click once more.

what is with that...? Other than this little thing, it works. I'd just like to know what's going on.

TIA!

We'll have to see your code again probably.