String (Hex) to Hex

[Mark Salsbery]

suguimoto wrote:

2147483647(dec) is equal to 7FFFFFFF(hex)

Yes

[Waldermort]

You are reading 32 characters from the file, is this correct? If so then declare your str array to hold 33 to allow for the null terminator. What is the value of endptr at the end of each loop?

[Mark Salsbery]

suguimoto wrote:

whats happening?

strtol() is returning LONG_MAX because of overflow. 0xFFFFFC24 is too big to fit in a signed long. Try strtoul()

[David Crow]

suguimoto wrote:

mem[i] = strtol(str, &endptr, 16);

Why aren't you using strtoul()?

---

"Approved Workmen Are Not Ashamed" - 2 Timothy 2:15

"Judge not by the eye but by the heart." - Native American Proverb

[suguimoto]

I changed to strtoul... and get the same result =/ also increased the size from 32 to 33, still get the same thing. very weird. But I'm still trying

[David Crow]

suguimoto wrote:

printf("%i\n\r", mem[i]);

Since mem[i] is an unsigned int, you should be using "%u" instead.

---

"Approved Workmen Are Not Ashamed" - 2 Timothy 2:15

"Judge not by the eye but by the heart." - Native American Proverb

[David Crow]

suguimoto wrote:

I changed to strtoul... and get the same result =/

How are you confirming that strtoul() is failing?

suguimoto wrote:

also increased the size from 32 to 33, still get the same thing.

Why would you expect that to make a difference, since there are no more than 10 characters per line?

---

"Approved Workmen Are Not Ashamed" - 2 Timothy 2:15

"Judge not by the eye but by the heart." - Native American Proverb

[suguimoto]

Thanks!! It's working! I used %x to display everything as Hex, and it's working perfectly. Thank you guys very much!!! code:

int main(int argc, char* argv[])
{
FILE *fp;
fp = fopen("mem.ini", "rt");
char str[32];
unsigned int mem[8192];
char * endptr;
int teste;

for(int i = 0; i<8192; i++){

fgets(str, 32, fp);

mem\[i\] = strtoul(str, &endptr, 16);

printf("%x\\n\\r", mem\[i\]); // %u   to display unsigned values
}

system("PAUSE");
return 0;
}

[David Crow]

suguimoto wrote:

for(int i = 0; i<8192; i++){

What if there are only 5, or 8193, lines in the file? It's not wise to hard-code such numbers.

---

"Approved Workmen Are Not Ashamed" - 2 Timothy 2:15

"Judge not by the eye but by the heart." - Native American Proverb

[suguimoto]

its must be huge. because it's simulating a memory in a "rom" inside a virtual processor. i needed a function "readmemh" from verilog, but it couldnt be translated to c++, so i did it manually. this mem is going to be read a lot of times and changed a lot of times. I think 8192 is fixed, as it was in Verilog. So i'm just gonna leave it like that Thanks!!