forums.ps2dev.org

[AnonymousTipster]

This seemingly should be trivial, but i'm trying to get a function to return multiple values. I've tried both approaches:

[Raphael]

Your first aproach is the correct one, with *widp/*heip as parameters, however since you gave the function pointers to the ints, you cannot change their values with widp = xx as this will only change the address of the pointer. You should use *widp = 10; *heip = 10; and it should work.

Your second aproach is illegal in every C/C++ compiler, as the & operator gives the address of a variable and this is always a defined value or 0 if the variable is not allocated. As such, this cannot be a parameter of a function (since the compiler would expect the addres to the variable in the parameter list, but at the time of declaration, this is not allocated or even defined).

Well, I'm rather bad at explaining complicated stuff like pointer functionality... but I hope you got the point :)

[memon]

The first piece of code propably crashes your program, and the second does not work because your compiling a .c file.... I guess :)

The reason the first one does not work is that it is overriding the address, not storing the value to the address pointed by the pointer. The first should be:

[AnonymousTipster]

Well, I replaced widp = 10; with *widp = 10; but that now crashes the PSP. That's still with .c by the way (it needs tweaking to compile in c++).

You call the function with &width, which passes a pointer to the variable, when the function recieves it with *widp, does it then revert it back into an integer, or is it still a pointer to an integer?

[jsgf]

It's still a pointer to an integer.

[Garak]

Hello,

Actually, Your second approach should work just fine.

unsigned char *loadPNGfromfile(const char *filename, int &widp, int &heip, int swizzleit) {widp = 10;heip=10;}

/*called like:*/
charstr = loadPNGfromfile("dir",width, height, 0); // added 0 as last param

You are just passing in your 2 ints by reference, so any changes to them made in your function will be reflected in main. However pass by reference was not around in straight C, so if you are using gcc to compile, you will get an error. In fact, you would need to make sure your file has a .cpp extenison, and use the C++ compiler to make that code compile correctly. But it is legal to pass values as you have done in the example above.

If that did crash, the likly culprit is that you are returning a pointer to a string you create locally within your function. Make sure you use new/malloc to allocate whatever it is you are returning as a string from your function. Otherwise, there is no telling what will actually be returned.

Garak

[AnonymousTipster]

I'd like to do this in C, because porting to C++ would mean changing the way i've setup function pointers and changing other things that C++ doesn't like.

Here is some source for C:

[jsgf]

Something else is going on. If you compile with "-O -Wall", do any warnings come up?

[Garak]

That function to convert a string to an integer is not correct. It should be as follows:

// implementation:
int parseInt(const char *str, bool *success)
{
const char *s = str;
int r = 0;
while (*s)
{
char c = *s;
if ((c < '0') || (c > '9'))
{
*success = false;
return 0;
}
r = r * 10 + (c - '0');
s++; // I am the missing piece. Without me you have an endless loop
}
*success = true;
return r;
}

What is the exact function you are having problems with, and can we get the complete function, prototype, and an example of you calling it? If the function is simply the one from your origional post, the C version should not compile, as you are not passing the correct number of paramaters to it in your sample call.

Garak

[AnonymousTipster]

Ok, well I spent the last half an hour isolating the function from my code, and i've managed to get it to work now (with the previous code snippet). I'm hoping that I just made a slip somewhere else, and it's not that 'compiler ram shuffle' problem that I had before.

Thanks for your help.