const declarations

Shine · Post by **Shine** » Sat Sep 17, 2005 4:45 pm

Some sce*-functions don't change the data, which are passed as pointers, like

int sceKernelUtilsMd5Digest(u8 *data, u32 size, u8 *digest);

so it should be

int sceKernelUtilsMd5Digest(const u8 *data, u32 size, u8 *digest);

The reason is that when I'm using 3'rd party libraries, sometimes the return values are const, which is fine, but then I have to cast it for the sce*-functions, which is not necessary. I don't think that the changes breaks code, but it makes new code more safe and readable.

Post by **mrbrown** » Sun Sep 18, 2005 12:17 am

And what's up with "u8 *"? It should be "void *". This isn't the K&R days :).

Shine · Post by **Shine** » Mon Sep 19, 2005 5:45 am

mrbrown wrote:And what's up with "u8 *"? It should be "void *". This isn't the K&R days :).

Yes, this would be fine, too.