Hello! I have a problem with memory (I think) so I tried to destroy the bitmaps with various commands, but I couldn't.

I received a lot of help from here:

http://cboard.cprogramming.com/c-programming/127048-error-expected-identifier-%91-%92-before-%91%7B%92-token-5.html

Please read that (the 5th page is the most important, but read the others if you want).

But now it's more Allegro-based problems, so I decided to post here.

My code:

The ideia is to make a kind of a slideshow... But when I get to this part:

(...)
bmp = load_bitmap("6.bmp", palette); 
   draw_sprite(screen, bmp, 0, 255);
     
      sleepTime=35;
   sleep(sleepTime);
   
    clear_bitmap(bmp);
   
   bmp = load_bitmap("Ubuntu.bmp",palette); 
   draw_sprite(screen, bmp, 1, 1); 
(...)

It crashes and exits. It displays the 6.bmp and does the sleeptime properly... But then it exits. I think it is the memory that is full, so it exits. It doesn't clear the bitmaps! I already tried with various and different commands, but it just don't works! The bitmaps aren't deleted!

Can you help me, please? When I thought I finished...

Thank you.

Regards,

JPCanaverde

May I ask... Where's the edit button?

EDIT:

Never mind, found it.

As I told, I tried with different commands. I tried that too, and happens the same.

Try checking load_bitmap for failure, if it returns NULL, it couldn't load the image, and draw_sprite will crash.

Please post the current state of your program. It seems like you made many adjustments to your code, so it's very hard to tell what's going wrong and why.

Ok, first of all:
Whenever you duplicate lines of code, what you really want is a function. Like this one:

You're also calling destroy_bitmap on a string in your example code, which is a bad idea.

You cannot run your ubunto executable in windows, because it's not made for windows, but for ubunto. While you can cross compile, I don't think that's what your're doing.

What class do you need this for? What's the assignment? I somehow doubt that your teacher requested you to write a slideshow program at the level that you demonstrate. If you can write a simpler program for the grade then I suggest you do so...

You seem to be a pretty big n00b for such a complicated program (by school standards). You don't even seem to comprehend subroutines and you're trying to use a third party library that you don't understand and are trying to cross-compile from Linux to Windows. It might be best to accept a failed grade and find yourself a C tutorial and start from scratch. The help you've been receiving seems to go right over your head.

There are many things wrong with your code. For starters, you must always check return values to ensure a called subroutine succeeded. If it didn't, continuing on might crash your program (which, surprise, is happening).

It's the first class of the morning of Wednesday.

The teacher didn't asked me to do... It is for Technological Education, and I proposed to, since we talked a bit about Binary Language. It's the final work for the year (I'm at the 7th, 12 years) and we were proposed to do what we wanted, practical or theoretical (mine counts as practical), but that had to do with the matter which had given.

As I had a little experience with C on PSP, i thought it would be fun to do it for PC.

If I need to, I compile Windows-Windows... And if it really has to be, I think I can do a extense "Hello World".

But, I really want to try. If I see I really can't, I give up and do the Hello World. But I wanna try. I love this stuff. In my holidays, I'll dedicate to C. But now I'm in a hurry...

Thank's for all help so far.

Ok, I'm passing something. I did what you told me, got this code:

And I get errors compiling!

I checked the code, and I think everything is "introduced". I don't get it...

Oh right. I have also no idea if that code compiles, since I "coded" it typing into the reply field.

If you find any bugs, you can keep them. They are yours!

Double check the filename (lower/upper case, spaces). Normally it's these small things that cost you a lot of time.

That's one of the reasons why using functions is a good thing: add a check once, get the message whenever it's used

Try using this slightly changed version of the function. You'll also have to
#include <stdio.h>

Since the loading code is now in a function, it's very easy to add an extra check (to see if the filename is the issue, or if it's a problem with the data itself).
I also removed the PALETTE, since it's not used anyway.

Oh, and I just wanted to repeat that you'll need to include stdio.h for this to work.

Of course your program is too slow, because you're waiting twice as long.

The function I wrote takes care of loading, displaying, waiting and destroying the bitmap.

Here's your code w/o the stuff that's not needed

The last number in each call is the sleep time. Which is in seconds. And 10 seconds are a long time. Even 3 seconds are pretty long when all you do is look at a screen.

See my (new) comment in the load_and_display function