syntax error : missing ')' before '{' and dynamic array in struct

[gazsux]

Im dynamically allocating some memory, but i get the error:
syntax error : missing ')' before '{'
I have counted the number of brackets and i cant see why im getting it. Any ideas? is there something im doing that im not allowed to do syntactically?

MALLOC(oObjectData, (sizeof(Objects *)*OINCREMENT));                   // Error here
for(iObjectSet=0; iObjectSet<OINCREMENT; iObjectSet++)
	MALLOC(oObjectData[iObjectSet], sizeof(Objects));      // and here

Also, related to this. The array im allocating is apart of a struct, so if i was to define the array. Will it then work if i do say:

struct_name.oObjectData[count]

Only if i make the array in the structure in the malloc function itself (eg MALLOC(struct_name.oObjectData, ...) it doesnt seem to like it.

[Vber]

What kind of 'malloc' is this? why you write it in uppercase? it's not the 'malloc' from stdlib, I'm quite sure, because the malloc from stdlib receives just one parameter, and to your MALLOC you're passing 2.

>>MALLOC(oObjectData[iObjectSet], sizeofObjects));
this shouldn't be:

MALLOC(oObjectData[iObjectSet], sizeofObjects);

or

MALLOC(oObjectData[iObjectSet], sizeof Objects);

I really don't understand if sizeofObjects is a variable, or it's a typo and you meant to write sizeof Objects, anyway, look there is an error here, you have one ')' that isn't needed.

[gazsux]

bah, sorry forgot this. I got it from a tutorial so my understanding of it isnt all it could be.

#define MALLOC(s,t) if(((s) = malloc(t)) == NULL) { oops("error: malloc()"); }

It just uses that to allow for an implicit error check

also i meant to put sizeof(Objects) where a variable name of a structure. Sorry, but i copied and pasted it frommy code, i dunno where the brackets went cos its in my source :S :$ i'll update my original accordingly

[Vber]

Well, looking again in your code:

MALLOC(oObjectData[iObjectSet], sizeof Objects);

And the second one, I'm not sure, but try something like that:

MALLOC(oObjectData, sizeof ((Objects *)*OINCREMENT));

[gazsux]

i think ill just break it back up into its component parts, ie/ use the real malloc. And maybe not run to boards so quickly after struggling to find an error... i figure it'll stop me from making such a prat of myself cheers for your help guys