hey i'm working on getting the basic components down for a game I am working on in my spare time, and I have just started doing keyboard initiated movement. However,if I open the task manager while the program is running, it shows CPU usage at 100%, and if I move my character up and down, my pagefile usage and overall memory usage slowly climb. Here's the main parts of my movement code:

Main (skipped over basic initialization code):

  while(!(key[KEY_ESC])){    //begin main game loop
    if (keypressed())
    {    
      animatePC(character_data,landFile,playerPos);
      clear_keybuf();
    }
  }

animatePC:

Doublebuffer Code:

void DrawDoublebuffer(BITMAP *buffer) {
    acquire_screen();
    blit(buffer, screen, 0, 0, 0, 0, buffer->w, buffer->h);
    release_screen();
}

and finally Drawmap

any help for fixing either of these problems would be greatly appreciated. as far as I can tell I'm destroying all bitmaps after they're used, so I dont know why the memory usage continues to climb every time I move in a direction. As for the 100% cpu, I'm assuming its cause I'm constantly checking for a keypress. if thats the reason for it, is there an easy way to fix this? any help would be greatly appreciated. thanks

And as for the memory leak, you never destroy "buff" in draw_map().

That said, don't recreate your buffers every time. It's horribly inefficient. Create them once at the start of the program and destroy them once at the end.

Also, look into separating your logic and your drawing...

[EDIT]
If I were you I'd dump the acquire and release as well when I was at it, as you're only doing one operation to the screen. Chances are that, one day, you'll be tempted to put something that doesn't belong there in between them... and then your program will die a gruesome death.

I believe your code has a great memory leak. See function animatePC. You create a buffer. But you only destroy it a couple of times inside for cycles. Your function should be like this instead:

Hehehe, the problem is that draw_map is creating bitmaps that are never being destroyed too. This code should work. Don't create a bitmap inside animatePC:

create_bitmap() is a signifacnt cost, only do it if you have to.
also ALWAYS check its return code, so if it actaully worked.