dont know how why this doesnt work

throws exception when trying to use pointer to store unsigned chars in array

Code:

---

unsigned char texture1 = 2700000;// 262 144;

---

the function :
I am stuck with not finding a solution now after trying different thing for a while

Code:

---

int readbmp() {       
        fileheader bmpheader;
        infoheader bmpinfoheader;
                unsigned char *p,*p2;
                p =&texture1;
               
                unsigned char header[100];
                int f, i, j,cnt;// buffer[10961024];
                cnt = 0;
        //char str[2000];
        unsigned char c;
        f = 2048;
       
        ifstream fileread("test.bmp",ios::binary);
        //ifstream fileread("test.txt");
        //read in 40 bytes header
        cout << "fileheaders";
        for (j = 0; j < (40+14); j++) {
                fileread >> c;
                header[j] = c;
                i = (int)c; i = i & 0xff;
                cout << i << "\n";
        }
        i = 0x001c;
        if (header[i] = 24) {
                cout << "24bit bmp ";
                cout << "bitmapdata";
                while (!fileread.eof()) {
                        fileread >> c;
                        (*p) = c;//exception always here
                        ++p;
                        //cout << c;
                       
                       
                        f--;
                        //if (f == 0) break;
                }


        }
        else {
                cout << "8bit palette bmp \n";
        }
        fileread.close();


        return 1;
}

---

I think that you've accidently posted c++ on the C forum

What is the maximum number that your system can store in an unsigned char?

unsigned char.... 8 bits value...

Quote:

---

Originally Posted by flp1969

unsigned char.... 8 bits value...

---

Then this isn't going to go well... :)

Code:

---

unsigned char texture1 = 2700000;

---

Code:

---

if (header[i] = 24)

---

This is an assignment, not a test

Code:

---

(*p) = c;//exception always here

++p;

---

Given @Click_here's point, I can't see how that would do anything else but fire an exception.

'p' points to something that stores one byte, somehow expected to store a value requiring 32 bits or more.

Maybe an array should be used?

* moved to C++ programming forum *

You should check the return value of fileread >> c rather than assume it succeeds. You can use it to control the loops. Do not use eof() to control reading in a loop like you are doing now since you would then continue in the loop body even if fileread >> c fails.

thanks everyone ,mistake to have declaration of texture1 far away and missed that
is it not prefered to have dynamic memory allocation with big files better and safer than arrays?
fileread or other function is possible to assign whole array after the 2 fileheaders are read in,instead of loop?

maybe possible to have two file opened simultanously and merge 8bit+24bit to 32bit?