Here are my variables most get destroyed at the end but sometimes the game ends in an error and I think either because of clicking or sounds but I do not know.

this is what I do at the end

I figure I should have more things destroyed but I am not sure but when I comment out unload_datafile(data) the problem is gone.

Currently the only thing destroyed from the data file is Background but that is not even used. background2 is the background I use for the game which is gotten from the datafile so nothing used is destroyed. Also when I comment out everything except the unload_datafile(data) I still get the error. Is unloading the datafile a need to kill all memory leaks?

The actuall error is a Windows XP type error send or dont send thing so all I get is the technical data in Hex that I cant copy. I will post anything that portrays to the problem I think is with ->

unload_datafile(data); //Seems to be where the problem is

This below is in a seperate header file where all the variables are assigned

DATAFILE *data;

This is in a header file where all the functions are assigned.

data = load_datafile("falling.dat");

And everything that uses the datafile so far.

draw_sprite(active_page,data[BACKGROUND2_BMP].dat,0,0);
.
play_sample((SAMPLE *)data[EXPLODE2_WAV].dat, volume, pan, pitch, 0);
.
play_sample((SAMPLE *)data[EXPLODE_WAV].dat, volume, pan, pitch, 0);

I will assume that is is what you mean

This is everything that is destroyed except background which is never actually used in the program only declared at program start. I hope this is what you meant.

Anything else and everything where the project is upto date is here FallingBallProject

I did the samething you did and only changed the lives to 1 ran the game a few times and everytime error after last screen where it would goto windows anyway so I decided to recompile with the unload_datafile(data); commented out and never once got the error. Don't know if it is even worth to unload the data file if I am getting that error.

well in FallingBall.h

BITMAP *explode[9];
BITMAP *ball[9];

is used setting up 9 balls

But you do not get the error by just letting the ball hit the end and watching the finish. And you can jsut press ESC instead of changing the lives but you actually have to fully interact try destroying one or 2 balls...Also setting ball speed in FallingBall.h and Rebuild all will make short work of everything.

Also what do you mean not just the definitions.

But I use the code in another file so I don't have to go searching through my entire program to find stuff. I guess in my own little way it is all organized in little drawers. When ever the entire file is all together it just looks to hecktic.

And I only just 5 mins ago changed it from Destroying 6 to destroying 9 problem still existed in both instances including commenting the destroying part alltogether.

Ok this is where the error happens.

After all lives are lost and the screen is cleared and you are showen you last score as shown here

textprintf_ex(screen,font,0,10,makecol(255,255,255),-1,"Last Score was %i Press ESC to Exit",score);
    while(!key[KEY_ESC]); //<---Something after here causes the error.   

Right when you press Escape to exit out completely this is where the error occurs.

and when I commented out everything that destroys it worked fine. So I decided to comment one out at a time and found thatunload_datafile(data);was the problem.

To further answer your question on how everything is placed. I started this program in QBASIC almost 8 years ago when I was 14. 3 years ago (about) I wanted to learn C/C++ and decided what a good way to learn by taking my favorite project from QBASIC and translating it to C/C++. About 2 years ago I set the project down and that was also the time I started using this forum. Then about a week ago I sat down looked at my code and was dumbfounded, didn't know what was going on so I just decided to start messing around with all the values and doing what I remembered and very quickly remembered just about everything.

Long story made short. It was how everything was done in QBASIC except in QBASIC I could use SUB's to make the program seem less cluttered. I know somethings dealing with my loops could be made considerably smaller, but I never really got into C/C++ because of the lack of graphic manipulation with out a library. And books on C/C++ never talked about those libraries. So instead of learning all of C/C++ I just learned how to put everything together and just get started in Allegro.

Not one that can read and learn. It must be hands on. And I got tired of talking about Printf and cout for 30 pages.;D

another note:
I thought that is what I was doing here. Although I dont see the need to have it in the code since the functions usually never are passing a value.

in FallingBall.h
void mousehold();
void dropball();
void moveball();
void movefloor();
void checkdead();
void getinput();
void setupscreen();
void startup();
void multicheck();
void singlecheck();
void exitall();
void setup();
void setupsound();

Hmm did a little tinker and added return 0; at the end. Problem solved. No more error on my machine. odd but my memory leak or something odd happens when the animation sequence is called on a large mass of balls.

was it atleast fun to play it 50 times I think I have played it in testing at least 1000+ times easy.

can you tell me if there is anything wrong with my grabber? Why I can't grab from the datafile. although BALL_BMP (102 x 17 BMP)

But here is the grabber

BITMAP *grabframe(BITMAP *source, int width, int height, int startx, int starty, int columns, int frame) 
{
           BITMAP *temp = create_bitmap(width,height);
           int x = startx + (frame ) * width;
           //int y = starty + (frame ) * height;
           blit(source,temp,x,0,0,0,width,height);
           return temp;
}

this is what I use the grabber for.

     temp = load_bitmap("ball.bmp",NULL);
     for (n = 0;n<6;n++)
         ball[n] = grabframe(temp,17,17,0,0,1,n);
     destroy_bitmap(temp);

is there a way to use this grabber for the datafile? It is basically like I wanted to do with the sound on another thread but just with a BMP that gets sections of it grabbed since the whole animation sequence is in one BMP
this is the output from the dat.exe file

- BMP  - BALL_BMP                     - bitmap (102x17, 16 bit)
correct to do what I want to do?

BITMAP *temp
is globally defined, but only used in loadsprites where it's already destroy_bitmap-ed : you shouldn't call destroy_bitmap(temp) "again" at the end of the program.

BITMAP *active_page is always a pointer pointing to buffer or buffer2: since there is already

destroy_bitmap(buffer);
destroy_bitmap(buffer2);

you shouldn't call destroy_bitmap(active_page)